jorgecarleitao commented on pull request #342:
URL: https://github.com/apache/arrow-datafusion/pull/342#issuecomment-842623668


   To be fair, it is fairly easy to enable that on the `BooleanBufferBuilder`; 
we need to add a method to set the bit using `bit_utils::set_bit`. `Buffer` for 
validity is effectively an immutable bitvec, and `BooleanBufferBuilder` is the 
corresponding mutable version; we just haven't added that functionality.
   
   I find the bitvec a bit overkill for a single use-case of a bitmap, 
specially when we already have a "bitvec" on arrow, but I am also fine pushing 
this through and leave this problem for later.


-- 
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:
[email protected]


Reply via email to