-
Notifications
You must be signed in to change notification settings - Fork 6.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
Make trash-to-DB size ratio limit configurable #3158
Make trash-to-DB size ratio limit configurable #3158
Conversation
Summary: Allow users to configure the trash-to-DB size ratio limit, so that ratelimits for deletes can be enforced even when larger portions of the database are being deleted. Test Plan: Reviewers: tec Subscribers: kolmike, logdevice-diffs@ Tasks: T22635202 Tags: logdevice Blame Revision:
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.
@gdavidsson has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
@gdavidsson has updated the pull request. View: changes, changes since last import |
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.
@gdavidsson has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
util/delete_scheduler.h
Outdated
assert(r >= 0); | ||
max_trash_db_ratio_ = r; | ||
return max_trash_db_ratio_.store(r); |
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.
I think return here is confusing
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.
LGTM
@gdavidsson has updated the pull request. View: changes, changes since last import |
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.
@gdavidsson has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Summary: Allow users to configure the trash-to-DB size ratio limit, so
that ratelimits for deletes can be enforced even when larger portions of
the database are being deleted.