tustvold commented on code in PR #3258:
URL: https://github.com/apache/arrow-rs/pull/3258#discussion_r1038396774
##########
arrow-data/src/bit_mask.rs:
##########
@@ -62,9 +65,42 @@ pub fn set_bits(
null_count as usize
}
+/// Combines the null bitmaps of multiple arrays using a bitwise `and`
operation.
+///
+/// This function is useful when implementing operations on higher level
arrays.
+pub fn combine_option_bitmap(
Review Comment:
Previously this would return an error if called with an empty `arrays`.
In practice no codepaths could hit this, and so I removed it as it seemed
unnecessary
--
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]
