XiangpengHao commented on PR #6624:
URL: https://github.com/apache/arrow-rs/pull/6624#issuecomment-2468403948

   >  switch RowSelection into an enum so we can dynamically switch
   
   That should work, my initial thoughts was to only use bitmap-based selectors 
when evaluating predicates (i.e., when filter pushdown is enabled); and always 
do range-based selectors otherwise.
   
   >  "why not just use BooleanArray directly?"
   
   I agree that most of the functionality overlaps with `BooleanBuffer` and can 
be implemented there. Will try to file a separate pr to extract the most common 
ones to `BooleanBuffer` in the next few days.


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