etseidl commented on PR #6257: URL: https://github.com/apache/arrow-rs/pull/6257#issuecomment-2366928233
> I go back and forth on this PR -- it isn't technically an API change in the sense of a breaking API change, but also it changes the content of written parquet files. Arguable the content better matches the parquet spec, I am just really worried about unintended consequences of doing this > > Maybe I am overly concerned. Then how about splitting this up? First change the read behavior so `None` isn't treated as `0`, keep the `missing_null_counts_as_zero` setting, but default to false. If anyone is broken by that behavior they can workaround by setting it `true`. After that's been around a release, then change the behavior on write. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
