-
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](cluster key) Cluster key skip non vertical compact #41413
base: master
Are you sure you want to change the base?
[fix](cluster key) Cluster key skip non vertical compact #41413
Conversation
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
run buildall |
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 41004 ms
|
TPC-DS: Total hot run time: 191675 ms
|
ClickBench: Total hot run time: 32.22 s
|
run p0 |
a8c8b5d
to
2a99934
Compare
run buildall |
TPC-H: Total hot run time: 40439 ms
|
TeamCity be ut coverage result: |
TPC-DS: Total hot run time: 191392 ms
|
ClickBench: Total hot run time: 31.64 s
|
2a99934
to
c964ec2
Compare
run buildall |
TeamCity be ut coverage result: |
c964ec2
to
3d39d0e
Compare
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TeamCity be ut coverage result: |
run performance |
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. |
now the cluster key does not support non vertical compaction.
#41208 try to implement it, but there are some problems, so we skip non vertical compaction for cluster key