-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Closed
Labels
arrowChanges to the arrow crateChanges to the arrow crateenhancementAny new improvement worthy of a entry in the changelogAny new improvement worthy of a entry in the changelog
Description
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
BooleanBufferBuilder currently has no mechanism to append an already packed bitmask, instead requiring bitwise iteration through the mask. This is particularly wasteful where the BooleanBufferBuilder has a length that is a multiple of 8, where a simple memcpy would suffice. I ran into this limitation whilst working on #1037
Describe the solution you'd like
In IOx we have a BitSet implementation that allows directly appending a bitmask - see here. This logic could pretty much just be copied and pasted into arrow.
Metadata
Metadata
Assignees
Labels
arrowChanges to the arrow crateChanges to the arrow crateenhancementAny new improvement worthy of a entry in the changelogAny new improvement worthy of a entry in the changelog