-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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](mtmv) regression test unstable and error #41145
base: master
Are you sure you want to change the base?
[fix](mtmv) regression test unstable and error #41145
Conversation
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
run buildall |
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: 41676 ms
|
TPC-DS: Total hot run time: 191859 ms
|
ClickBench: Total hot run time: 32.68 s
|
run buildall |
TPC-H: Total hot run time: 40878 ms
|
TPC-DS: Total hot run time: 191885 ms
|
ClickBench: Total hot run time: 32.83 s
|
run buildall |
1 similar comment
run buildall |
TPC-H: Total hot run time: 41151 ms
|
TPC-DS: Total hot run time: 190767 ms
|
ClickBench: Total hot run time: 32.79 s
|
run buildall |
TPC-H: Total hot run time: 41199 ms
|
TPC-DS: Total hot run time: 192372 ms
|
ClickBench: Total hot run time: 33.04 s
|
run buildall |
TPC-H: Total hot run time: 40793 ms
|
TPC-DS: Total hot run time: 193087 ms
|
ClickBench: Total hot run time: 32.03 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
} | ||
|
||
// multi mv part in rewrite process, any of them rewrte success without check if chosen by cbo or not | ||
def mv_rewrite_any_success_without_check_chosen = { query_sql, mv_names, sync_cbo_rewrite = true -> |
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.
It can automatically adjust the switch according to the version number?
run buildall |
TPC-H: Total hot run time: 40877 ms
|
TPC-DS: Total hot run time: 192216 ms
|
ClickBench: Total hot run time: 32.55 s
|
… mv and fix test not exist
be158f5
to
621dc13
Compare
run buildall |
TPC-H: Total hot run time: 40749 ms
|
TPC-DS: Total hot run time: 192644 ms
|
ClickBench: Total hot run time: 32.33 s
|
Proposed changes
This is brought by #39265
MaterializedViewRewriteSuccessButNotChose
, because in these tests materialized view are not analyzed after create, So Addanalyze clause
when waite materialized view task finishedcheck_mv_rewrite_success
is not available in Suite.groovy. Fix this by using correct method.