mohitgurav20 commented on PR #25811: URL: https://github.com/apache/datafusion/pull/25811#issuecomment-6832484202
@alamb Added a two-row-group test to cover the interleaved case you mentioned. rg0 has a in [10,20], rg1 has a in [30,40]. With WHERE a = 30 AND b = 300, the snapshot shows column a's Bloom filter gets fetched for both groups first, rg0 gets pruned after that, and then column b's Bloom filter is only requested for rg1. So rg0 never triggers a range read for column b at all — that's the skipping behavior pinned in the snapshot. Also scoped BloomColumnGuarantee and the related functions to pub(crate) since nothing outside the crate needs them, and removed record_bloom_filter_matched — it was just wrapping a single method call. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
