alamb opened a new issue #911:
URL: https://github.com/apache/arrow-rs/issues/911


   **Is your feature request related to a problem or challenge? Please describe 
what you are trying to do.**
   Re #817 ; As @jhorstmann  pointed out on #810 at 
https://github.com/apache/arrow-rs/pull/810/files#r742905920
   
   `ArrayData::try_new()` function includes an optional `null_count` argument 
and a validity buffer. If these numbers differ,  wrong results can occur (and 
maybe also undefined behavior) in some later operation. 
   
   Note that as part of #817 we will be validating that the bitmaps are 
consistent with the declared `null_count`
   
   **Describe the solution you'd like**
   
   Since most callers pass `None` anyway at which point we calculate the 
number, if we simply removed the option to pass in an inconsistent value we 
would avoid a class of inconsistencies I would suggest to remove the 
`null_count` parameter from `try_new` to completely avoid of inconsistencies.
   
   **Describe alternatives you've considered**
   N/A 
   
   **Additional context**
   Add any other context or screenshots about the feature request here.
   


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