HyukjinKwon opened a new pull request, #48724:
URL: https://github.com/apache/arrow/pull/48724

   ### Rationale for this change
   
   
https://github.com/apache/arrow/commit/d0f3b5f3c74f67c7ca941c98bd60148e9a9e94f0 
introduced the todo:
   
   
https://github.com/apache/arrow/blob/727106f7ff65065298e1e79071fed2a408b4b4d6/cpp/src/arrow/compute/kernels/vector_selection_test.cc#L518
   
   which I presumed that the author forgot to address this.
   
   ### What changes were included in this PR?
   
   - Made `CompareAndFilter` helper functions null-aware by checking validity 
bitmap
   - Updated filter kernel tests to use random null probabilities (0.0 to 1.0)
   
   ### Are these changes tested?
   
   Yes, new tests were added to address nulls, and locally tested.
   
   ### Are there any user-facing changes?
   
   No, test-only.


-- 
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]

Reply via email to