-
Notifications
You must be signed in to change notification settings - Fork 3.6k
[Fix](job) fix drop db error and duplicate fragment_id when retry #57307
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
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
ClickBench: Total hot run time: 28.03 s |
|
run buildall |
ClickBench: Total hot run time: 27.97 s |
|
run buildall |
TPC-DS: Total hot run time: 190868 ms |
|
run p0 |
|
run cloud_p0 |
|
run nonConcurrent |
|
run feut |
ClickBench: Total hot run time: 27.67 s |
|
run p0 |
FE Regression Coverage ReportIncrement line coverage |
FE Regression Coverage ReportIncrement line coverage |
|
run cloud_p0 |
FE Regression Coverage ReportIncrement line coverage |
|
run buildall |
TPC-DS: Total hot run time: 188155 ms |
ClickBench: Total hot run time: 27.77 s |
FE UT Coverage ReportIncrement line coverage |
|
run buildall |
ClickBench: Total hot run time: 28.01 s |
|
run buildall |
TPC-DS: Total hot run time: 187289 ms |
ClickBench: Total hot run time: 27.76 s |
|
run buildall |
ClickBench: Total hot run time: 27.78 s |
FE Regression Coverage ReportIncrement line coverage |
...-core/src/main/java/org/apache/doris/job/extensions/insert/streaming/StreamingInsertJob.java
Outdated
Show resolved
Hide resolved
FE Regression Coverage ReportIncrement line coverage |
|
run buildall |
ClickBench: Total hot run time: 28.34 s |
FE Regression Coverage ReportIncrement line coverage |
|
run buildall |
TPC-DS: Total hot run time: 190159 ms |
ClickBench: Total hot run time: 27.55 s |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
PR approved by at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
FE Regression Coverage ReportIncrement line coverage |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…7307) ### What problem does this PR solve? Fixed the following issues: 1. In a cloud environment, if the database has been deleted, attempting to drop a job would result in an error. 2. In a cloud environment, MS offsets were not successfully put, and statistics were inaccurate. 3. When a StreamingTask timed out and retried, a duplicate framement error would be reported.
…en retry #57307 (#57375) Cherry-picked from #57307 Co-authored-by: wudi <[email protected]>
What problem does this PR solve?
Fixed the following issues:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)