Skip to content

Conversation

@andygrove
Copy link
Member

…ateMode (#4878)

  • Fix column indices in EnforceDistribution optimizer in Partial AggregateMode

Column expressions need to be updated to correspond with the partial aggregation schema rather than the input schema.

  • Simplify new_group_by calculation

Which issue does this PR close?

Closes #.

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

…ateMode (apache#4878)

* Fix column indices in EnforceDistribution optimizer in Partial AggregateMode

Column expressions need to be updated to correspond with the partial aggregation schema rather than the input schema.

* Simplify new_group_by calculation
@github-actions github-actions bot added the core Core DataFusion crate label Jan 17, 2023
@andygrove andygrove closed this Jan 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Core DataFusion crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants