alamb commented on issue #6215:
URL: https://github.com/apache/arrow-rs/issues/6215#issuecomment-2408974240

   > thanks @alamb @Michael-J-Ward for changing the `null_count` to 
`null_count_opt`, but iiuc the ambiguity still exists on `Some(0)` given the 
following code??
   > 
   > 
https://github.com/apache/arrow-rs/blob/master/parquet/src/file/statistics.rs#L129-L142
   
   You are correct @jp0317  -- we have a PR up that proposes changing this 
behavior too:  https://github.com/apache/arrow-rs/pull/6257 but I have been 
worried about unintended consequences of merging it 


-- 
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]

Reply via email to