ARROW-16646: [C++] Allow key columns to be scalars in Bloom filter #13236
Conversation
westonpace
left a comment
There was a problem hiding this comment.
Thanks for the fix. I suppose (very) long term we will want to include some kind of support for RLE arrays in the lightweight array (KeyColumnArray) utilities?
|
Yes, long-term we'd want some general way to deal with dictionary/RLE/any other encoding we'd care about later on. That is for the long term, and a good candidate for this would be in the pipeline Driver class where we'll add later. |
|
Benchmark runs are scheduled for baseline = b7edf64 and contender = 5c02620. 5c02620 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
No description provided.