ch-sc commented on a change in pull request #9965:
URL: https://github.com/apache/arrow/pull/9965#discussion_r610595476
##########
File path: rust/arrow/src/compute/kernels/boolean.rs
##########
@@ -628,6 +642,16 @@ mod tests {
}
}
+ #[test]
+ fn test_boolean_array_kleene_no_remainder() {
+ let n = 1024;
+ let a = BooleanArray::from(vec![true; n]);
+ let b = BooleanArray::from(vec![None; n]);
+ let result = or_kleene(&a, &b).unwrap();
Review comment:
This test aims to reproduce the bug. There are a number of unit tests
(`test_bool_array_and_kleene_nulls`, `test_bool_array_or_kleene_nulls`,
`test_binary_boolean_kleene_kernel`) that cover different cases of bits in the
remainder word. Note that if `batch size < 64` all bits are stored in the
remainder chunk.
--
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
