search

apache / incubator-gluten

Gluten is a middle layer responsible for offloading JVM-based SQL engines' execution to native engines.
https://gluten.apache.org/
Apache License 2.0
1.22k stars 438 forks source link

[GLUTEN-8005][VL] Add MergeTwoPhasesHashBaseAggregate to injectRas list #8006

Closed yikf closed 4 days ago

yikf commented 4 days ago

What changes were proposed in this pull request?

This PR aims to add MergeTwoPhasesHashBaseAggregate to injectRas list to make RAS support merge aggregate, and enable some UTs related to merge aggregate.

How was this patch tested?

Exist UTs.

This follows https://github.com/apache/incubator-gluten/pull/7752.

github-actions[bot] commented 4 days ago

https://github.com/apache/incubator-gluten/issues/8005

github-actions[bot] commented 4 days ago

Run Gluten Clickhouse CI on x86

github-actions[bot] commented 4 days ago

Run Gluten Clickhouse CI on x86

yikf commented 4 days ago

@zhztheplayer @jackylee-ch Could you please take a look, thanks

github-actions[bot] commented 4 days ago

Run Gluten Clickhouse CI on x86

zhztheplayer commented 4 days ago

@yikf should we also remove this line, thanks!

yikf commented 4 days ago

@yikf should we also remove this line, thanks!

yes, thanks

github-actions[bot] commented 4 days ago

Run Gluten Clickhouse CI on x86

github-actions[bot] commented 4 days ago

Run Gluten Clickhouse CI on x86

github-actions[bot] commented 4 days ago

Run Gluten Clickhouse CI on x86

github-actions[bot] commented 4 days ago

Run Gluten Clickhouse CI on x86