apache / datafusion

Apache DataFusion SQL Query Engine
https://datafusion.apache.org/
Apache License 2.0
6.37k stars 1.2k forks source link

Remove unreachable filter logic in final grouping stage #13463

Closed jayzhan211 closed 6 days ago

jayzhan211 commented 1 week ago

Which issue does this PR close?

Closes #.

Rationale for this change

Aggregate Filter could be applied in partial stage, at the point we reach final stage we don't need to handle filter. Therefore, to simplify the code logic, I remove related unreachable code

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

jayzhan211 commented 6 days ago

Thanks @alamb @findepi