Skip to content

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

Merged
BiteTheDDDDt merged 1 commit into
apache:masterfrom
BiteTheDDDDt:fix_0905
Sep 6, 2024
Merged

[Bug](map) fix wrong result on map_agg with streaming agg#40454
BiteTheDDDDt merged 1 commit into
apache:masterfrom
BiteTheDDDDt:fix_0905

Conversation

@BiteTheDDDDt

Copy link
Copy Markdown
Contributor

Proposed changes

fix wrong result on map_agg with streaming agg

@BiteTheDDDDt

Copy link
Copy Markdown
ContributorAuthor

run buildall

@mrhhsgmrhhsg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Sep 6, 2024
@github-actions

Copy link
Copy Markdown
Contributor

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

@github-actions

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@BiteTheDDDDt
BiteTheDDDDt merged commit 44b42b9 into apache:masterSep 6, 2024
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
@gavinchougavinchou mentioned this pull request Feb 18, 2025
HappenLee pushed a commit to HappenLee/incubator-doris that referenced this pull request Apr 24, 2026
## Proposed changes
fix wrong result on map_agg with streaming agg
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approvedIndicates a PR has been approved by one committer.dev/2.1.6-mergeddev/3.0.3-mergedp0_wreviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@BiteTheDDDDt@mrhhsg@Mryange@gavinchou@yiguolei@dataroaring