Skip to content

Improve aggregation documentation for multi-phase aggregation - #11695

Closed
alamb wants to merge 2 commits into
apache:mainfrom
alamb:alamb/aggregate_docs
Closed

Improve aggregation documentation for multi-phase aggregation #11695
alamb wants to merge 2 commits into
apache:mainfrom
alamb:alamb/aggregate_docs

Conversation

@alamb

Copy link
Copy Markdown
Contributor

Draft as it builds on #11627

Which issue does this PR close?

Part of #6937

Rationale for this change

As part of reviewing @korowa 's wonderful #11627 to improve multi-phase aggregation I want to document how it works a bit more (so it is easier to add the relevant support for other aggregates)

What changes are included in this PR?

  1. Update docstrings with additional information about multi-phase aggregation

Are these changes tested?

By docs CI

Are there any user-facing changes?

Just docs, no functional changes

@github-actionsgithub-actionsBot added documentation Improvements or additions to documentation logical-expr Logical plan and expressions physical-expr Changes to the physical-expr crates core Core DataFusion crate sqllogictest SQL Logic Tests (.slt) labels Jul 28, 2024
@alamb
alambforce-pushed the alamb/aggregate_docs branch from afc91de to 1d96e25CompareJuly 28, 2024 12:50
@github-actionsgithub-actionsBot removed physical-expr Changes to the physical-expr crates core Core DataFusion crate labels Jul 28, 2024
@alamb
alambforce-pushed the alamb/aggregate_docs branch from e151786 to 92d2485CompareJuly 28, 2024 13:42
@alamb

alamb commented Aug 4, 2024

Copy link
Copy Markdown
ContributorAuthor

Per korowa#172 (comment)

This was merged into #11627 with b3c033f

@alambalamb closed this Aug 4, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentationImprovements or additions to documentationlogical-exprLogical plan and expressionssqllogictestSQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@alamb@korowa