-
Notifications
You must be signed in to change notification settings - Fork 780
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
Let blocks_cleaner delete blocks concurrently #5028
Let blocks_cleaner delete blocks concurrently #5028
Conversation
db57caa
to
3ef1731
Compare
PS: In fact, for parameters like this, I think the default value is enough. Exposing this parameter will only increase the burden. |
+1... lets try to keep the config simple |
Yes. If most people agree, I will remove the parameter control, the default value is 16. |
7643fe1
to
e51fc41
Compare
Signed-off-by: wangguoliang <iamwgliang@gmail.com>
e51fc41
to
0a69b1c
Compare
@yeya24 is it ready to be merged ? |
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.
Thanks!
Signed-off-by: wangguoliang iamwgliang@gmail.com
What this PR does:
Let blocks_cleaner delete blocks concurrently. And it can make blocks cleanup faster when there is large amount of blocks.
Which issue(s) this PR fixes:
Fixes #
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]