Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug](map) fix wrong result on map_agg with streaming agg #40454

Merged
merged 1 commit into from
Sep 6, 2024

Conversation

BiteTheDDDDt
Copy link
Contributor

Proposed changes

fix wrong result on map_agg with streaming agg

@BiteTheDDDDt
Copy link
Contributor Author

run buildall

Copy link
Member

@mrhhsg mrhhsg left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Sep 6, 2024
Copy link
Contributor

github-actions bot commented Sep 6, 2024

PR approved by at least one committer and no changes requested.

Copy link
Contributor

github-actions bot commented Sep 6, 2024

PR approved by anyone and no changes requested.

@BiteTheDDDDt BiteTheDDDDt merged commit 44b42b9 into apache:master Sep 6, 2024
23 of 29 checks passed
BiteTheDDDDt added a commit that referenced this pull request Oct 8, 2024
fix wrong result on map_agg with streaming agg
dataroaring pushed a commit that referenced this pull request Oct 9, 2024
## Proposed changes
fix wrong result on map_agg with streaming agg
BiteTheDDDDt added a commit that referenced this pull request Jan 2, 2025
…41536)

fix wrong result on map_agg with streaming agg

## Proposed changes
pick from part of #40454
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by one committer. dev/2.1.6-merged dev/3.0.3-merged p0_w reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants