Uh oh!
There was an error while loading. Please reload this page.
[fix](cloud) release fd after clear file cache - #50555
Conversation
Thearas
commented
Apr 29, 2025
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
originally, fd is still hold by FDCache after clear file cache (sync) will cause disk space never reaching zero Signed-off-by: zhengyu <zhangzhengyu@selectdb.com>
6fdf686 to
f4ccae6Comparefreemandealer
commented
Apr 29, 2025
run buildall |
doris-robot
commented
Apr 29, 2025
TPC-H: Total hot run time: 34183 ms |
doris-robot
commented
Apr 29, 2025
TPC-DS: Total hot run time: 192162 ms |
doris-robot
commented
Apr 29, 2025
ClickBench: Total hot run time: 28.59 s |
doris-robot
commented
Apr 29, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
Apr 29, 2025
BE Regression P0 && UT Coverage ReportIncrement line coverage Increment coverage report
|
| int64_t disposible_queue_size = _disposable_queue.get_elements_num(cache_lock); | ||
| int64_t ttl_queue_size = _ttl_queue.get_elements_num(cache_lock); | ||
| if (config::clear_fd_after_clear_file_cache) { |
There was a problem hiding this comment.
no need an extra config, just release them directly after files are deleted.
There was a problem hiding this comment.
in case of performance and robust regression. we can remove this config if it's stable.
There was a problem hiding this comment.
it seems there is no performance problem, line 2069 auto s = _storage->clear(clear_msg); is blocking and time-consuming job before removing FDs.
to avoid too many knobs, keep it simple if the behavior is determined.
we have too many knobs for file cache, too many.
freemandealer
commented
Apr 30, 2025
run buildall |
doris-robot
commented
Apr 30, 2025
TPC-H: Total hot run time: 33926 ms |
doris-robot
commented
Apr 30, 2025
TPC-DS: Total hot run time: 184650 ms |
doris-robot
commented
Apr 30, 2025
ClickBench: Total hot run time: 29.28 s |
hello-stephen
commented
Apr 30, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
Apr 30, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
freemandealer
commented
Apr 30, 2025
run cloud_p0 |
Signed-off-by: zhengyu <zhangzhengyu@selectdb.com>
freemandealer
commented
May 7, 2025
run buildall |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
doris-robot
commented
May 7, 2025
TPC-H: Total hot run time: 34534 ms |
doris-robot
commented
May 7, 2025
TPC-DS: Total hot run time: 192804 ms |
doris-robot
commented
May 7, 2025
ClickBench: Total hot run time: 29.14 s |
hello-stephen
commented
May 7, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
May 7, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
freemandealer
commented
May 8, 2025
run buildall |
doris-robot
commented
May 8, 2025
TPC-H: Total hot run time: 34153 ms |
doris-robot
commented
May 8, 2025
TPC-DS: Total hot run time: 186465 ms |
doris-robot
commented
May 8, 2025
ClickBench: Total hot run time: 28.88 s |
hello-stephen
commented
May 8, 2025
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
hello-stephen
commented
May 8, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
freemandealer
commented
May 13, 2025
run external |
freemandealer
commented
May 13, 2025
run p0 |
freemandealer
commented
May 13, 2025
run external |
hello-stephen
commented
May 13, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
1 similar comment
hello-stephen
commented
May 13, 2025
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
originally, fd is still hold by FDCache after clear file cache (sync) will cause disk space never reaching zero Signed-off-by: zhengyu <zhangzhengyu@selectdb.com>
originally, fd is still hold by FDCache after clear file cache (sync) will cause disk space never reaching zero Signed-off-by: zhengyu <zhangzhengyu@selectdb.com>
originally, fd is still hold by FDCache after clear file cache (sync) will cause disk space never reaching zero
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)