Remove redundant Aggregate when `DISTINCT` & `GROUP BY` are in the same query#11781

Merged

ozankabak merged 31 commits intoapache:mainfrom synnada-ai:feature/eliminate-aggregate

Aug 4, 2024

+119-30

Commits on Feb 22, 2024

Commits on Feb 23, 2024

Merge branch 'apache:main' into main
mustafasrepo
authored

Commits on Feb 26, 2024

Commits on Feb 27, 2024

Commits on Feb 28, 2024

Merge branch 'apache:main' into main
mustafasrepo
authored

Commits on Mar 1, 2024

Merge branch 'apache:main' into main
mustafasrepo
authored

Commits on Mar 5, 2024

Merge branch 'apache:main' into main
mustafasrepo
authored

Commits on Aug 1, 2024

initialize eliminate_aggregate.rs rule
Mert Akkaya
committed
Merge remote-tracking branch 'refs/remotes/origin/apache_main' into feature/eliminate-aggregate
Mert Akkaya
committed
remove redundant prints
Mert Akkaya
committed

Commits on Aug 2, 2024

Add multiple group by expression handling.
mustafasrepo
committed
rename eliminate_aggregate.rs as eliminate_distinct.rs
Mert Akkaya
committed
Merge remote-tracking branch 'refs/remotes/origin/apache_main' into feature/eliminate-aggregate
Mert Akkaya
committed
remove logic for distinct on since group by statement must exist in projection
Mert Akkaya
committed
format code
Mert Akkaya
committed
add eliminate_distinct rule to tests
Mert Akkaya
committed
simplify function
Mert Akkaya
committed
fix child issue
Mert Akkaya
committed
format
Mert Akkaya
committed
fix docs
Mert Akkaya
committed
remove eliminate_distinct rule and make it a part of replace_distinct_aggregate
Mert Akkaya
committed
Update datafusion/common/src/functional_dependencies.rs

mertak-synnada
and
ozankabak
authored
add comment and fix variable call
Mert Akkaya
committed
fix test cases as optimized plan
Mert Akkaya
committed
format code
Mert Akkaya
committed
simplify comments

mertak-synnada
and
ozankabak
authored
do not replace redundant distincts with aggregate
Mert Akkaya
committed
merge from remote
mertak-synnada
committed