Skip to content

[VL] Eager aggregation: Push group-only aggregation - #13254

Draft
zhztheplayer wants to merge 4 commits into
apache:mainfrom
zhztheplayer:wip-eageragg-distinct
Draft

zhztheplayer wants to merge 4 commits into
apache:mainfrom
zhztheplayer:wip-eageragg-distinct

Conversation

@zhztheplayer

Copy link
Copy Markdown
Member

This PR pushes grouping-only aggregation below both sides of an inner equi-join to reduce duplicate rows entering the join.

For example:

SELECT DISTINCT l.name, r.category
FROM l JOIN r ON l.id = r.id;

Before:
Distinct [name, category]
  Join [l.id = r.id]
    l
    r

After:
Distinct [name, category]
  Join [l.id = r.id]
    Distinct [id, name]
      l
    Distinct [id, category]
      r

The lower aggregates retain the join keys, and the top DISTINCT preserves the original result.

@github-actions github-actions Bot added the CORE works for Gluten Core label Oct 12, 2026
@github-actions

Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@zhztheplayer
zhztheplayer force-pushed the wip-eageragg-distinct branch from d7224a2 to f47c867 Compare October 15, 2026 09:58
@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 github-actions Bot added the VELOX label Oct 15, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CORE works for Gluten Core VELOX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant