mohitgurav20 commented on PR #25811: URL: https://github.com/apache/datafusion/pull/25811#issuecomment-6506802619
@alamb Opened #26897 with the baseline test against `main`. It records the current 2-request pattern where both filters are fetched even when an earlier column rejects. Once that merges, rebasing this PR will show `requests2.len()` dropping from 2 to 1 in that test. Also pushed the refactoring here: - Guarantees and per-column evaluation are moved into `RowGroupAccessPlanFilter`, so opener only handles async loading. - Removed `access_plan_mut()`. - Dropped the criterion bench. -- 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]
