Skip to content

[GLUTEN-9137][CH] Support CollectLimit for CH backend#9139

Merged
exmy merged 8 commits intoapache:mainfrom
exmy:gluten-9137
Apr 25, 2025
Merged

[GLUTEN-9137][CH] Support CollectLimit for CH backend#9139
exmy merged 8 commits intoapache:mainfrom
exmy:gluten-9137

Conversation

@exmy
Copy link
Copy Markdown
Contributor

@exmy exmy commented Mar 26, 2025

What changes were proposed in this pull request?

Support CollectLimit operator for CH backend based #8914

(Fixes: #9137)

How was this patch tested?

(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)

(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

@github-actions github-actions bot added CORE works for Gluten Core VELOX CLICKHOUSE labels Mar 26, 2025
@github-actions
Copy link
Copy Markdown

#9137

@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

1 similar comment
@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

2 similar comments
@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@exmy
Copy link
Copy Markdown
Contributor Author

exmy commented Apr 23, 2025

@jinchengchenghh Could you help review these velox changes? Thanks!

@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

Copy link
Copy Markdown
Contributor

@jinchengchenghh jinchengchenghh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approve for the velox backend

@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

Copy link
Copy Markdown
Contributor

@taiyang-li taiyang-li left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@exmy exmy merged commit 2f13452 into apache:main Apr 25, 2025
49 checks passed
lwz9103 added a commit that referenced this pull request Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLICKHOUSE CORE works for Gluten Core VELOX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CH] Support CollectLimit

3 participants