search

apache / datafusion-comet

Apache DataFusion Comet Spark Accelerator
https://datafusion.apache.org/comet
Apache License 2.0
823 stars 163 forks source link

chore: Remove ObjectHashAggregate from BaseAggregateExec pattern as it is not supported #990

Closed viirya closed 1 month ago

viirya commented 1 month ago

Which issue does this PR close?

Closes #.

Rationale for this change

It is uncertain if DataFusion hash aggregate operator can replace Spark ObjectHashAggregate. We should disable it from Comet planner until we make sure it works.

What changes are included in this PR?

How are these changes tested?

viirya commented 1 month ago

Close in favor of #991