-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[fix](mtmv) regression test unstable and error #41145
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
Conversation
|
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
|
run buildall |
morningman
left a comment
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 |
b6dfe2f to
b8e2b54
Compare
|
run buildall |
TPC-H: Total hot run time: 41207 ms |
TPC-DS: Total hot run time: 192045 ms |
ClickBench: Total hot run time: 33.14 s |
|
run buildall |
TPC-H: Total hot run time: 41613 ms |
TPC-DS: Total hot run time: 191220 ms |
ClickBench: Total hot run time: 33.08 s |
|
run p0 10 |
|
PR approved by at least one committer and no changes requested. |
zfr9527
left a comment
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
This is brought by apache#39265 1. Some test fail because `MaterializedViewRewriteSuccessButNotChose`, because in these tests materialized view are not analyzed after create, So Add `analyze clause` when waite materialized view task finished 2. No signature of method: org.apache.doris.regression.suite.Suite.check_mv_rewrite_success(), because `check_mv_rewrite_success` is not available in Suite.groovy. Fix this by using correct method. 3. If partition row count statistics is not ready, should not check mv rewrte by CBO succesfully
This is brought by apache#39265 1. Some test fail because `MaterializedViewRewriteSuccessButNotChose`, because in these tests materialized view are not analyzed after create, So Add `analyze clause` when waite materialized view task finished 2. No signature of method: org.apache.doris.regression.suite.Suite.check_mv_rewrite_success(), because `check_mv_rewrite_success` is not available in Suite.groovy. Fix this by using correct method. 3. If partition row count statistics is not ready, should not check mv rewrte by CBO succesfully
Proposed changes
This is brought by #39265
MaterializedViewRewriteSuccessButNotChose, because in these tests materialized view are not analyzed after create, So Addanalyze clausewhen waite materialized view task finishedcheck_mv_rewrite_successis not available in Suite.groovy. Fix this by using correct method.