etk18 opened a new pull request, #20064: URL: https://github.com/apache/datafusion/pull/20064
## Which issue does this PR close? Closes #20053 ## Rationale for this change `ArrowBytesViewMap` previously used `Vec<bool>` to track null values. This PR replaces it with `BooleanBufferBuilder`, which is significantly more memory-efficient and faster, and aligns with Apache Arrow best practices for building validity bitmaps. This change improves performance and memory usage without changing behavior. ## What changed - Replaced `Vec<bool>` with `BooleanBufferBuilder` for null tracking - Updated null buffer construction to use Arrow-native buffers - Kept ordering and semantics unchanged ## Tests - `cargo test -p datafusion-physical-expr-common` > Note: Full workspace tests require `protoc`; this PR was validated with the affected crate tests. https://github.com/user-attachments/assets/88b1af34-a905-43bd-b9e9-065858d0781d -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
