haohuaijin opened a new pull request, #10574: URL: https://github.com/apache/arrow-rs/pull/10574
# Which issue does this PR close? Split out of #10446 so that CI can compare both sides — the benchmark is new there, so the merge-base has nothing to compare against and the `main` column comes out empty. # Rationale for this change `row_selector.rs` already benchmarks `intersection`/`union`, but it builds the operands with `from_filters`, which is selector-backed. Those take the `RowSelector` merge path and never reach the bitwise one used when both operands are mask-backed. # What changes are included in this PR? Adds `mask_intersection`/`mask_union`, varying the two dimensions that drive the bitwise path: - the ratio between operand lengths, since unequal lengths pass the longer side's tail through unchanged - the bit offsets the operands carry, since masks come from `BooleanBuffer::slice` and whether the two share a sub-64-bit alignment decides which path the underlying helpers take Benchmark only, no library changes. # Are these changes tested? N/A — this is a benchmark. It builds and runs on `main` as-is. # Are there any user-facing changes? No. -- 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]
