-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
[improve](mow) merge and remove old version of delete bitmap when cumulative compaction is done #42479
base: branch-2.1
Are you sure you want to change the base?
[improve](mow) merge and remove old version of delete bitmap when cumulative compaction is done #42479
Conversation
…ulative compaction is done
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
run buildall |
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.
clang-tidy made some suggestions
|
||
#pragma once | ||
|
||
#include <stdint.h> |
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.
warning: inclusion of deprecated C++ header 'stdint.h'; consider using 'cstdint' instead [modernize-deprecated-headers]
#include <stdint.h> | |
#include <cstdint> |
private: | ||
Status _handle_show_delete_bitmap_count(HttpRequest* req, std::string* json_result); | ||
|
||
private: |
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.
warning: redundant access specifier has the same accessibility as the previous access specifier [readability-redundant-access-specifiers]
private: |
Additional context
be/src/http/action/delete_bitmap_action.h:44: previously declared here
private:
^
TeamCity be ut coverage result: |
pick pr #40204 to branch 2.1