-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[Enhancement] Improve Compaction Profiling and Logging-task2 #42227
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. Since 2024-03-18, the Document has been moved to doris-website. |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
Yukang-Lian
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.
plz add some regression tests
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
run buildall |
1 similar comment
|
run buildall |
|
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 41424 ms |
TPC-DS: Total hot run time: 191918 ms |
ClickBench: Total hot run time: 33.27 s |
|
run buildall |
TPC-DS: Total hot run time: 192472 ms |
ClickBench: Total hot run time: 33.01 s |
83623c4 to
d0851ab
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
|
run buildall |
|
clang-tidy review says "All clean, LGTM! 👍" |
5feb863 to
869252e
Compare
|
run buildall |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 40982 ms |
TPC-DS: Total hot run time: 196980 ms |
ClickBench: Total hot run time: 32.61 s |
fix some bug
### What problem does this PR solve? Issue Number: close #xxx Related PR: #42227 Problem Summary: Issue Number: close #41753 Add detailed parameters to compaction status: add last cumulative failure time last base failure time last full failure time last cumulative success time last base success time last full success time last cumulative schedule time last base schedule time last full schedule time last cumulative status last base status last full status in cloud_tablet.cpp and add last cumulative schedule time last full schedule time last cumulative status last full status in tablet.cpp Co-authored-by: yoruet <1559650411@qq.com>
### What problem does this PR solve? Issue Number: close #xxx Related PR: #42227 Problem Summary: Issue Number: close #41753 Add detailed parameters to compaction status: add last cumulative failure time last base failure time last full failure time last cumulative success time last base success time last full success time last cumulative schedule time last base schedule time last full schedule time last cumulative status last base status last full status in cloud_tablet.cpp and add last cumulative schedule time last full schedule time last cumulative status last full status in tablet.cpp Co-authored-by: yoruet <1559650411@qq.com>
…#50950) ### What problem does this PR solve? Issue Number: close #xxx Related PR: apache#42227 Problem Summary: Issue Number: close apache#41753 Add detailed parameters to compaction status: add last cumulative failure time last base failure time last full failure time last cumulative success time last base success time last full success time last cumulative schedule time last base schedule time last full schedule time last cumulative status last base status last full status in cloud_tablet.cpp and add last cumulative schedule time last full schedule time last cumulative status last full status in tablet.cpp Co-authored-by: yoruet <1559650411@qq.com>
…#50950) ### What problem does this PR solve? Issue Number: close #xxx Related PR: apache#42227 Problem Summary: Issue Number: close apache#41753 Add detailed parameters to compaction status: add last cumulative failure time last base failure time last full failure time last cumulative success time last base success time last full success time last cumulative schedule time last base schedule time last full schedule time last cumulative status last base status last full status in cloud_tablet.cpp and add last cumulative schedule time last full schedule time last cumulative status last full status in tablet.cpp Co-authored-by: yoruet <1559650411@qq.com>
|
We're closing this PR because it hasn't been updated in a while. |
Proposed changes
Issue Number: close #41753

Add detailed parameters to compaction status:
add
in
cloud_tablet.cppand add
in
tablet.cpp