Skip to content

[Fix](Streamingjob) Fix the issue of synchronization failure under empty tables - #59735

Merged
JNSimba merged 2 commits into
apache:masterfrom
JNSimba:fix-empty-table
Jan 12, 2026
Merged

[Fix](Streamingjob) Fix the issue of synchronization failure under empty tables#59735
JNSimba merged 2 commits into
apache:masterfrom
JNSimba:fix-empty-table

Conversation

@JNSimba

@JNSimbaJNSimba commented Jan 9, 2026

Copy link
Copy Markdown
Member

What problem does this PR solve?

Issue Number: close #xxx

Related PR: #58898

Release note

None

Check List (For Author)

  • Test

    • Regression test
    • Unit Test
    • Manual test (add detailed scripts or steps below)
    • No need to test or manual test. Explain why:
      • This is a refactor/code format and no logic has been changed.
      • Previous test can cover this change.
      • No code files have been changed.
      • Other reason

  • Behavior changed:

    • No.
    • Yes.

  • Does this need documentation?

    • No.
    • Yes.

Check List (For Reviewer who merge this PR)

  • Confirm the release note
  • Confirm test cases
  • Confirm document
  • Add branch pick label

@hello-stephen

hello-stephen commented Jan 9, 2026

Copy link
Copy Markdown
Contributor

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

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@JNSimba

JNSimba commented Jan 9, 2026

Copy link
Copy Markdown
MemberAuthor

run buildall

dataroaring
dataroaring previously approved these changes Jan 10, 2026
View reviewed changes

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

github-actionsBot commented Jan 10, 2026

Copy link
Copy Markdown
Contributor

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

@github-actions

github-actionsBot commented Jan 10, 2026

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@github-actionsgithub-actionsBot removed the approved Indicates a PR has been approved by one committer. label Jan 12, 2026
@JNSimba

JNSimba commented Jan 12, 2026

Copy link
Copy Markdown
MemberAuthor

run buildall

@JNSimba

JNSimba commented Jan 12, 2026

Copy link
Copy Markdown
MemberAuthor

run cloud_p0

@JNSimba

JNSimba commented Jan 12, 2026

Copy link
Copy Markdown
MemberAuthor

run p0

2 similar comments
@JNSimba

JNSimba commented Jan 12, 2026

Copy link
Copy Markdown
MemberAuthor

run p0

@JNSimba

JNSimba commented Jan 12, 2026

Copy link
Copy Markdown
MemberAuthor

run p0

sollhui
sollhui approved these changes Jan 12, 2026
View reviewed changes

@sollhuisollhui 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-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Jan 12, 2026
@github-actions

github-actionsBot commented Jan 12, 2026

Copy link
Copy Markdown
Contributor

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

@JNSimba
JNSimba merged commit 7919f2a into apache:masterJan 12, 2026
28 of 29 checks passed
github-actionsBot pushed a commit that referenced this pull request Jan 12, 2026
@JNSimbaYour Name
[Fix](Streamingjob) Fix the issue of synchronization failure under em…
4b8a2ac
…pty tables (#59735)
### What problem does this PR solve?
Fix the issue of synchronization failure under empty tables Related PR: #58898
@github-actionsgithub-actionsBot mentioned this pull request Jan 12, 2026
Merged
yiguolei pushed a commit that referenced this pull request Jan 13, 2026
@github-actions@JNSimba
branch-4.0: [Fix](Streamingjob) Fix the issue of synchronization fail…
a33cc78
…ure under empty tables #59735 (#59790)
Cherry-picked from #59735
Co-authored-by: wudi <wudi@selectdb.com>
zzzxl1993 pushed a commit to zzzxl1993/doris that referenced this pull request Jan 13, 2026
@JNSimba@zzzxl1993
[Fix](Streamingjob) Fix the issue of synchronization failure under em…
c813744
…pty tables (apache#59735)
### What problem does this PR solve?
Fix the issue of synchronization failure under empty tables Related PR: apache#58898
@yiguoleiyiguolei mentioned this pull request Jan 23, 2026
Closed
ybtsdst pushed a commit to ybtsdst/doris that referenced this pull request Feb 27, 2026
@github-actions@JNSimbabingtao.yin
branch-4.0: [Fix](Streamingjob) Fix the issue of synchronization fail…
7212fcd
…ure under empty tables apache#59735 (apache#59790)
Cherry-picked from apache#59735
Co-authored-by: wudi <wudi@selectdb.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@CalvinKirsCalvinKirsCalvinKirs approved these changes
@sollhuisollhuisollhui approved these changes
@dataroaringdataroaringdataroaring left review comments

Assignees

No one assigned

Labels

approvedIndicates a PR has been approved by one committer.dev/4.0.3-mergedreviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@JNSimba@hello-stephen@CalvinKirs@sollhui@dataroaring@yiguolei