Skip to content

[Bug](runtime-filter) fix wrong check on BloomFilterFuncBase::merge - #41670

Merged
BiteTheDDDDt merged 1 commit into
apache:masterfrom
BiteTheDDDDt:fix_1010_2
Oct 10, 2024
Merged

[Bug](runtime-filter) fix wrong check on BloomFilterFuncBase::merge#41670
BiteTheDDDDt merged 1 commit into
apache:masterfrom
BiteTheDDDDt:fix_1010_2

Conversation

@BiteTheDDDDt

@BiteTheDDDDtBiteTheDDDDt commented Oct 10, 2024

Copy link
Copy Markdown
Contributor

Proposed changes

fix wrong check on BloomFilterFuncBase::merge
introduced by #41648

@hello-stephen

Copy link
Copy Markdown
Contributor

skip buildall

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Oct 10, 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.

@github-actions

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@BiteTheDDDDt
BiteTheDDDDt merged commit a9caf05 into apache:masterOct 10, 2024
cjj2010 pushed a commit to cjj2010/doris that referenced this pull request Oct 12, 2024
…pache#41670)
## Proposed changes
fix wrong check on BloomFilterFuncBase::merge
introduced by apache#41648
HappenLee pushed a commit to HappenLee/incubator-doris that referenced this pull request Apr 24, 2026
…pache#41670)
## Proposed changes
fix wrong check on BloomFilterFuncBase::merge
introduced by apache#41648
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.reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@BiteTheDDDDt@hello-stephen@zhangstar333