keirsalterego commented on issue #6528: URL: https://github.com/apache/arrow-rs/issues/6528#issuecomment-3726863154
That makes sense. Defining with_nulls as the union of the new and existing null buffers (i.e., an intersection of validity) completely avoids the safety hazards of unmasking undefined data, and it does so without introducing a new panic condition. This fits the primary use case (combining a filter/mask) perfectly. I will implement this using NullBuffer::union for PrimitiveArray, BooleanArray, and GenericByteArray. -- 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]
