Uh oh!
There was an error while loading. Please reload this page.
[workloadgroup](improvement) using min/max cpu and memory percent to unify soft and hard limit - #54243
Conversation
Thearas
commented
Aug 3, 2025
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
yiguolei
commented
Aug 5, 2025
run buildall |
doris-robot
commented
Aug 5, 2025
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
| {"MIN_CPU_PERCENT", TYPE_BIGINT, sizeof(int64_t), true}, | ||
| {"MAX_CPU_PERCENT", TYPE_BIGINT, sizeof(int64_t), true}, | ||
| {"MIN_MEMORY_PERCENT", TYPE_BIGINT, sizeof(int64_t), true}, | ||
| {"MAX_MEMORY_PERCENT", TYPE_BIGINT, sizeof(int64_t), true}, |
There was a problem hiding this comment.
In an 1024G memoy's node, 1% means 10G memory, 0.5% means 5G memory, it's not a small waste , maybe double is better.
There was a problem hiding this comment.
I think it is ok. If the memory is very large, use do not need care about 5GB memory.
But this is a bug, the type should be varchar, not int64.
Uh oh!
There was an error while loading. Please reload this page.
| int64_t _memory_limit; // bytes | ||
| std::atomic<int> _min_cpu_percent = 0; | ||
| std::atomic<int> _max_cpu_percent = 100; | ||
| bool _enable_memory_overcommit = true; |
There was a problem hiding this comment.
_enable_memory_overcommit is useless?
There was a problem hiding this comment.
Currently it is useful, we only use it to skip memory check....
Uh oh!
There was an error while loading. Please reload this page.
yiguolei
commented
Aug 6, 2025
run buildall |
hello-stephen
commented
Aug 6, 2025
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
Uh oh!
There was an error while loading. Please reload this page.
| .add(MAX_REMOTE_SCAN_THREAD_NUM).add(MIN_REMOTE_SCAN_THREAD_NUM) | ||
| .add(MEMORY_LOW_WATERMARK).add(MEMORY_HIGH_WATERMARK) | ||
| .add(MAX_CPU_PERCENT).add(MIN_CPU_PERCENT) | ||
| .add(MAX_MEMORY_PERCENT).add(MIN_MEMORY_PERCENT).add(ENABLE_MEMORY_OVERCOMMIT) |
There was a problem hiding this comment.
ENABLE_MEMORY_OVERCOMMIT这个是不是可以删了
There was a problem hiding this comment.
还不太行,现在老版本升级过来的normal的内存只有40%,之前是软限,所以没问题。 我们现在都改成硬限了,会导致query 被cancel。所以还得让默认的enable memory overcommit 发挥作用,让他是true
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
doris-robot
commented
Aug 6, 2025
TPC-H: Total hot run time: 34160 ms |
doris-robot
commented
Aug 6, 2025
TPC-DS: Total hot run time: 170687 ms |
doris-robot
commented
Aug 6, 2025
ClickBench: Total hot run time: 35.23 s |
doris-robot
commented
Aug 6, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
7e91ce9 to
ad316d3Compareyiguolei
commented
Aug 7, 2025
run feut |
yiguolei
commented
Aug 7, 2025
run feut |
yiguolei
commented
Aug 9, 2025
run buildall |
hello-stephen
commented
Aug 9, 2025
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
Aug 9, 2025
FE UT Coverage ReportIncrement line coverage |
doris-robot
commented
Aug 9, 2025
TPC-H: Total hot run time: 33850 ms |
doris-robot
commented
Aug 9, 2025
TPC-DS: Total hot run time: 170582 ms |
doris-robot
commented
Aug 9, 2025
ClickBench: Total hot run time: 35.05 s |
doris-robot
commented
Aug 9, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
Aug 9, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
yiguolei
commented
Aug 10, 2025
run p0 |
yiguolei
commented
Aug 10, 2025
run p0 |
1 similar comment
yiguolei
commented
Aug 10, 2025
run p0 |
yiguolei
commented
Aug 10, 2025
run buildall |
doris-robot
commented
Aug 10, 2025
Cloud UT Coverage ReportIncrement line coverage Increment coverage report
|
doris-robot
commented
Aug 10, 2025
TPC-H: Total hot run time: 33618 ms |
doris-robot
commented
Aug 10, 2025
TPC-DS: Total hot run time: 169837 ms |
doris-robot
commented
Aug 10, 2025
ClickBench: Total hot run time: 34.99 s |
hello-stephen
commented
Aug 10, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
Aug 10, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
Aug 10, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
Aug 10, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
Aug 11, 2025
FE UT Coverage ReportIncrement line coverage |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
Uh oh!
There was an error while loading. Please reload this page.
What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)