-
Notifications
You must be signed in to change notification settings - Fork 508
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
[feat]curvefs/metaserver: add delete inode monitor #2822
[feat]curvefs/metaserver: add delete inode monitor #2822
Conversation
6b1964a
to
1e63a9c
Compare
cicheck |
1 similar comment
cicheck |
@@ -151,6 +166,7 @@ class S3ClientAdaptorImpl : public S3ClientAdaptor { | |||
uint64_t batchSize_; | |||
uint32_t objectPrefix_; | |||
bool enableBatchDelete_; | |||
std::unique_ptr<S3ClientAdaptorMetric> metric_; |
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.
no need to save metric as a pointer, just S3ClientAdaptorMetric metric_
?
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.
no need to save metric as a pointer, just
S3ClientAdaptorMetric metric_
?
fix
1e63a9c
to
c61ac70
Compare
cicheck |
3 similar comments
cicheck |
cicheck |
cicheck |
c61ac70
to
472930d
Compare
cicheck |
Signed-off-by: Cyber-SiKu <Cyber-SiKu@outlook.com>
472930d
to
21d2d4c
Compare
cicheck |
4 similar comments
cicheck |
cicheck |
cicheck |
cicheck |
What problem does this PR solve?
Issue Number: #xxx
Problem Summary:
What is changed and how it works?
What's Changed:
How it Works:
Side effects(Breaking backward compatibility? Performance regression?):
Check List