You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Filter pushdown past children with fetch limits causes correctness bug #21063
In the push_down_filter optimisation rule, we currently don't check if the child of a filter node has fetch limits. This is semantically equivalent to pushing a filter past a limit node, which is wrong. The same issue exists in the gather_filters_for_pushdown implementation for SortExec.
To Reproduce
I added unit tests in the fix PR which reproduce this behaviour.
Expected behavior
This should never happen, pushing a filter past for example a sort with a fetch limit causes incorrect behaviour.
Activity
shivbhatia10 commentedon Mar 19, 2026
take
shivbhatia10 commentedon Mar 19, 2026
#21057
Fix push_down_filter for children with non-empty fetch fields (#21057)
Fix push_down_filter for children with non-empty fetch fields (apache…
Fix push_down_filter for children with non-empty fetch fields (apache…
Fix push_down_filter for children with non-empty fetch fields (apache…
Fix push_down_filter for children with non-empty fetch fields (apache…
Fix push_down_filter for children with non-empty fetch fields (apache…
Fix push_down_filter for children with non-empty fetch fields (apache…
Fix push_down_filter for children with non-empty fetch fields (apache…
Fix push_down_filter for children with non-empty fetch fields (apache…
[branch-53] Fix push_down_filter for children with non-empty fetch fi…
[branch-52] Fix push_down_filter for children with non-empty fetch fi…