-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
[opt](memory) Optimize ClearCacheAction
implementation
#38438
Conversation
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 39816 ms
|
TPC-DS: Total hot run time: 173290 ms
|
ClickBench: Total hot run time: 31.02 s
|
be/src/common/config.cpp
Outdated
@@ -133,13 +133,11 @@ DEFINE_mString(process_full_gc_size, "20%"); | |||
// If false, cancel query when the memory used exceeds exec_mem_limit, same as before. | |||
DEFINE_mBool(enable_query_memory_overcommit, "true"); | |||
|
|||
DEFINE_mBool(disable_memory_gc, "false"); | |||
DEFINE_mBool(enable_memory_reclamation, "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.
这些配置文件的名字先别改了,影响不大。另外,改了还得改文档,各个版本之间兼容还有问题。
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.
done
87bebf2
to
0220155
Compare
0220155
to
2e5ac71
Compare
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
2 similar comments
clang-tidy review says "All clean, LGTM! 👍" |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 42065 ms
|
TPC-DS: Total hot run time: 169380 ms
|
ClickBench: Total hot run time: 29.89 s
|
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 41835 ms
|
TPC-DS: Total hot run time: 170311 ms
|
ClickBench: Total hot run time: 30.3 s
|
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 41408 ms
|
TPC-DS: Total hot run time: 169187 ms
|
ClickBench: Total hot run time: 30.05 s
|
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
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
1. `ClearCacheAction` support for specifying cache, `http://beip:8040/api/clear_cache/all`, `http://beip:8040/api/clear_cache/DataPageCache` 2. Optimize result returned by `ClearCacheAction`. 3. Optimize Allocator large memory printing stacktrace 4. `cache_prune_stale_thread` no longer check conf to prune cache, which has delays and brings a bad experience.
1. `ClearCacheAction` support for specifying cache, `http://beip:8040/api/clear_cache/all`, `http://beip:8040/api/clear_cache/DataPageCache` 2. Optimize result returned by `ClearCacheAction`. 3. Optimize Allocator large memory printing stacktrace 4. `cache_prune_stale_thread` no longer check conf to prune cache, which has delays and brings a bad experience.
1. `ClearCacheAction` support for specifying cache, `http://beip:8040/api/clear_cache/all`, `http://beip:8040/api/clear_cache/DataPageCache` 2. Optimize result returned by `ClearCacheAction`. 3. Optimize Allocator large memory printing stacktrace 4. `cache_prune_stale_thread` no longer check conf to prune cache, which has delays and brings a bad experience.
ClearCacheAction
support for specifying cache,http://beip:8040/api/clear_cache/all
,http://beip:8040/api/clear_cache/DataPageCache
ClearCacheAction
.cache_prune_stale_thread
no longer check conf to prune cache, which has delays and brings a bad experience.