-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[Fix](group commit) Fix table not found fault when disable group commit #39731
[Fix](group commit) Fix table not found fault when disable group commit #39731
Conversation
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
run buildall |
TPC-H: Total hot run time: 38130 ms
|
TPC-DS: Total hot run time: 196699 ms
|
ClickBench: Total hot run time: 30.93 s
|
run buildall |
TPC-H: Total hot run time: 38381 ms
|
TPC-DS: Total hot run time: 197938 ms
|
ClickBench: Total hot run time: 32.1 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. |
run buildall |
TPC-H: Total hot run time: 38029 ms
|
TPC-DS: Total hot run time: 192189 ms
|
ClickBench: Total hot run time: 30.31 s
|
...t/suites/load_p0/stream_load/test_group_commit_stream_load_with_nonexist_db_and_table.groovy
Outdated
Show resolved
Hide resolved
run buildall |
TPC-H: Total hot run time: 37670 ms
|
TPC-DS: Total hot run time: 192254 ms
|
ClickBench: Total hot run time: 30.45 s
|
run buildall |
TPC-H: Total hot run time: 37945 ms
|
TPC-DS: Total hot run time: 191480 ms
|
ClickBench: Total hot run time: 30.7 s
|
run buildall |
TPC-H: Total hot run time: 37516 ms
|
TPC-DS: Total hot run time: 191475 ms
|
ClickBench: Total hot run time: 30.68 s
|
run buildall |
TPC-H: Total hot run time: 37737 ms
|
TPC-DS: Total hot run time: 192231 ms
|
ClickBench: Total hot run time: 31 s
|
run feut |
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. |
run feut |
…it (#39731) ## Proposed changes <!--Describe your changes.--> selectRedirectBackend param table id is only needed by group commit. In non group commit mode, if table id is null, we need to ignore it.
…it (apache#39731) <!--Describe your changes.--> selectRedirectBackend param table id is only needed by group commit. In non group commit mode, if table id is null, we need to ignore it.
…it (apache#39731) <!--Describe your changes.--> selectRedirectBackend param table id is only needed by group commit. In non group commit mode, if table id is null, we need to ignore it.
Proposed changes
selectRedirectBackend param table id is only needed by group commit. In non group commit mode, if table id is null, we need to ignore it.