Skip to content

AggExec: implement columnar accumulator states.#646

Merged
lihao712 merged 1 commit intomasterfrom
dev-agg-columnar-acc
Nov 21, 2024
Merged

AggExec: implement columnar accumulator states.#646
lihao712 merged 1 commit intomasterfrom
dev-agg-columnar-acc

Conversation

@richox
Copy link
Contributor

@richox richox commented Nov 11, 2024

Which issue does this PR close?

Closes #645 .

  1. supports columnar accumulator, implements AccColumn trait replacing the old AccumStateRow.
  2. partial skipping optimization, never spill in partial aggregation.
  3. execution framework refactor, introduce ExecutionContext to simplify operator implmentations.

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

@richox richox force-pushed the dev-agg-columnar-acc branch 5 times, most recently from 92ccae9 to 1b30cc9 Compare November 14, 2024 17:38
@richox richox changed the title AggExec: implement columnar accumulator states. WIP: AggExec: implement columnar accumulator states. Nov 19, 2024
@richox richox force-pushed the dev-agg-columnar-acc branch from 1b30cc9 to deffd25 Compare November 20, 2024 07:12
@richox richox changed the title WIP: AggExec: implement columnar accumulator states. AggExec: implement columnar accumulator states. Nov 20, 2024
@richox richox force-pushed the dev-agg-columnar-acc branch from deffd25 to 0bcc695 Compare November 20, 2024 17:11
@richox richox force-pushed the dev-agg-columnar-acc branch from 0bcc695 to 24d4e1f Compare November 20, 2024 18:46
@lihao712 lihao712 merged commit 9a10161 into master Nov 21, 2024
richox added a commit that referenced this pull request Nov 25, 2024
refactor execution context

Co-authored-by: zhangli20 <zhangli20@kuaishou.com>
richox added a commit that referenced this pull request Nov 29, 2024
refactor execution context

Co-authored-by: zhangli20 <zhangli20@kuaishou.com>
@richox richox mentioned this pull request Dec 6, 2024
@cxzl25 cxzl25 deleted the dev-agg-columnar-acc branch September 5, 2025 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

implement columnar accumulator states

2 participants