Skip to content

[fix](move-memtable) fix auto partition load issue caused by #42039 - #42484

Merged
liaoxin01 merged 1 commit into
apache:masterfrom
kaijchen:fix-42039
Oct 25, 2024
Merged

[fix](move-memtable) fix auto partition load issue caused by #42039#42484
liaoxin01 merged 1 commit into
apache:masterfrom
kaijchen:fix-42039

Conversation

@kaijchen

@kaijchenkaijchen commented Oct 25, 2024

Copy link
Copy Markdown
Member

Proposed changes

Fix incremental streams is not closed, a bug introduced in #42039

@doris-robot

doris-robot commented Oct 25, 2024

Copy link
Copy Markdown

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@kaijchen

kaijchen commented Oct 25, 2024

Copy link
Copy Markdown
MemberAuthor

run buildall

@github-actions

github-actionsBot commented Oct 25, 2024

Copy link
Copy Markdown
Contributor

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

@kaijchenkaijchen changed the title [fix](move-memtable) fix auto partition problems caused by #42039[fix](move-memtable) fix auto partition load issue caused by #42039Oct 25, 2024
kaijchen added a commit to kaijchen/doris that referenced this pull request Oct 25, 2024
@kaijchen
[fix](move-memtable) fix auto partition load issue caused byapache#4…
a4dad9f
…2039 (apache#42484)
kaijchen added a commit to kaijchen/doris that referenced this pull request Oct 25, 2024
@kaijchen
[fix](move-memtable) fix auto partition load issue caused byapache#4…
8d56512
…2039 (apache#42484)
@kaijchenkaijchen mentioned this pull request Oct 25, 2024
Merged
liaoxin01
liaoxin01 approved these changes Oct 25, 2024
View reviewed changes

@liaoxin01liaoxin01 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@github-actions

github-actionsBot commented Oct 25, 2024

Copy link
Copy Markdown
Contributor

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

@github-actionsgithub-actionsBot added approved Indicates a PR has been approved by one committer. reviewed labels Oct 25, 2024
@github-actions

github-actionsBot commented Oct 25, 2024

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@kaijchenkaijchen mentioned this pull request Oct 25, 2024
Merged
@doris-robot

doris-robot commented Oct 25, 2024

Copy link
Copy Markdown

TeamCity be ut coverage result:
Function Coverage: 37.46% (9713/25932)
Line Coverage: 28.71% (80540/280547)
Region Coverage: 28.13% (41642/148018)
Branch Coverage: 24.69% (21156/85688)
Coverage Report: http://coverage.selectdb-in.cc/coverage/5f0ed9c20c8de116400767ee3ecc749cb3971341_5f0ed9c20c8de116400767ee3ecc749cb3971341/report/index.html

@liaoxin01
liaoxin01 merged commit b9a287e into apache:masterOct 25, 2024
yiguolei pushed a commit that referenced this pull request Oct 25, 2024
@kaijchen
[fix](move-memtable) fix auto partition load issue caused by#42039(#…
83b81c1
…42484) (#42486)
backport #42484
liaoxin01 pushed a commit that referenced this pull request Oct 26, 2024
@kaijchen
[fix](move-memtable) fix auto partition load issue caused by#42039(#…
fe0da72
…42484) (#42487)
backport #42484
HappenLee pushed a commit to HappenLee/incubator-doris that referenced this pull request Apr 24, 2026
@kaijchen@HappenLee
[fix](move-memtable) fix auto partition load issue caused byapache#4…
600f48e
…2039 (apache#42484)
Fix incremental streams is not closed, a bug introduced in apache#42039
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@liaoxin01liaoxin01liaoxin01 approved these changes
@zclllyybbzclllyybbzclllyybb approved these changes

Assignees

No one assigned

Labels

approvedIndicates a PR has been approved by one committer.dev/2.1.7-mergeddev/3.0.3-mergedp0_breviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@kaijchen@doris-robot@liaoxin01@zclllyybb@yiguolei