Ref #94. It might be better for the output to say something like

added: foo (binary file)

than the current

Binary files /dev/null and b/foo differ

Environment

Delta-rs version: all

Binding: rust

Bug

What happened:

Some of the fields are incorrectly marked as optional in action.rs

What you expected to happen:

Make sure optionality annotation matches with what's in delta spec and reference implementation: https://github.com/delta-io/delta/blob/master/core/src/main/scala/org/apache/spark/sql/delta/a

We should allow the user to configure these parameters, especially for the approximate Firth fit. cc @bboutkov since I believe this came up in your tests.

logistic_regression definition: https://github.com/projectglow/glow/blob/master/python/glow/gwas/log_reg.py#L24-L33

Our stats logging code are written as self.log_message_buffered(msg, state.value_buffers.len()).await, which blocks the current task from execution until the stats send async function has completed. It's better to handle these kind of instrumentation io in a separate tokio task in the background.