Uh oh!
There was an error while loading. Please reload this page.
[fix](cloud-mow) MS should delete the existing keys before rewriting it when processing old version delete bitmap on cu compaction - #42379
Conversation
doris-robot
commented
Oct 24, 2024
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
hust-hhb
commented
Oct 24, 2024
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
doris-robot
commented
Oct 24, 2024
TPC-H: Total hot run time: 41147 ms |
doris-robot
commented
Oct 24, 2024
TeamCity be ut coverage result: |
doris-robot
commented
Oct 24, 2024
TPC-DS: Total hot run time: 192399 ms |
doris-robot
commented
Oct 24, 2024
ClickBench: Total hot run time: 32.27 s |
Uh oh!
There was an error while loading. Please reload this page.
hust-hhb
commented
Oct 24, 2024
run buildall |
doris-robot
commented
Oct 24, 2024
TPC-H: Total hot run time: 41566 ms |
doris-robot
commented
Oct 24, 2024
TeamCity be ut coverage result: |
doris-robot
commented
Oct 24, 2024
TPC-DS: Total hot run time: 191310 ms |
doris-robot
commented
Oct 24, 2024
ClickBench: Total hot run time: 32.43 s |
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.
| << " total_size=" << total_size << " unlock=" << unlock; | ||
| } | ||
| void MetaServiceImpl::update_delete_bitmap2(google::protobuf::RpcController* controller, |
There was a problem hiding this comment.
warning: function 'update_delete_bitmap2' exceeds recommended size/complexity thresholds [readability-function-size]
voidMetaServiceImpl::update_delete_bitmap2(google::protobuf::RpcController* controller,
^Additional context
cloud/src/meta-service/meta_service.cpp:1858: 132 lines including whitespace and comments (threshold 80)
voidMetaServiceImpl::update_delete_bitmap2(google::protobuf::RpcController* controller,
^Uh oh!
There was an error while loading. Please reload this page.
hust-hhb
commented
Oct 31, 2024
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
hust-hhb
commented
Oct 31, 2024
run buildall |
1 similar comment
hust-hhb
commented
Oct 31, 2024
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
1 similar comment
clang-tidy review says "All clean, LGTM! 👍" |
doris-robot
commented
Oct 31, 2024
TeamCity be ut coverage result: |
…itmap when doing cu compaction
clang-tidy review says "All clean, LGTM! 👍" |
doris-robot
commented
Nov 1, 2024
TPC-DS: Total hot run time: 197711 ms |
doris-robot
commented
Nov 1, 2024
ClickBench: Total hot run time: 32.38 s |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
hust-hhb
commented
Nov 1, 2024
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
doris-robot
commented
Nov 1, 2024
TPC-H: Total hot run time: 41684 ms |
doris-robot
commented
Nov 1, 2024
TeamCity be ut coverage result: |
doris-robot
commented
Nov 1, 2024
TPC-DS: Total hot run time: 196504 ms |
doris-robot
commented
Nov 1, 2024
ClickBench: Total hot run time: 32.66 s |
PR approved by at least one committer and no changes requested. |
…it when processing old version delete bitmap on cu compaction (apache#42379) pr apache#40204 support deleting old version delete bitmap when doing cu compaction, it will update delete bitmap with agg result, then deleting old version delete bitmap. Updating delete bitmap means overwriting existing keys, however delete bitmap may split into multiple kvs to store in fdb, so we should delete the existing keys and rewrite it to fdb instead of overwriting directly.
pr #40204 support deleting old version delete bitmap when doing cu compaction, it will update delete bitmap with agg result, then deleting old version delete bitmap. Updating delete bitmap means overwriting existing keys, however delete bitmap may split into multiple kvs to store in fdb, so we should delete the existing keys and rewrite it to fdb instead of overwriting directly.