rluvaton commented on PR #9970: URL: https://github.com/apache/arrow-rs/pull/9970#issuecomment-4470203029
> Thanks @rluvaton @gstvg before expanding arrow-rs lets try to scope the concern. > > Ideally to include in the PR description the explanation why this problem is happening, what is the reason of ArrayRef require cleaning, because under usual circumstances it should not. When we discussed the change in [apache/datafusion#22158 (comment)](https://github.com/apache/datafusion/pull/22158#discussion_r3241321436) I feel the scope was to explore if `BooleanBuffer` can calculate `has_false` without adding another `BooleanBuilder` wrapper on top of it This was before the discussion to move the `has_false`, but also this supported map and byte based array and not only lists this can happen under normal circumstances, for example using `nullif` kernel, this is not a problem it is a valid case, updated the description. -- 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]
