-
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
Add enable_split_merge option for CompressedSecondaryCache #10690
Conversation
@gitbw95 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@gitbw95 has updated the pull request. You must reimport the pull request before landing. |
2 similar comments
@gitbw95 has updated the pull request. You must reimport the pull request before landing. |
@gitbw95 has updated the pull request. You must reimport the pull request before landing. |
@gitbw95 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
1 similar comment
@gitbw95 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
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
@gitbw95 has updated the pull request. You must reimport the pull request before landing. |
1 similar comment
@gitbw95 has updated the pull request. You must reimport the pull request before landing. |
@gitbw95 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
fae873f
to
5b685f5
Compare
@gitbw95 has updated the pull request. You must reimport the pull request before landing. |
5b685f5
to
04ceb66
Compare
@gitbw95 has updated the pull request. You must reimport the pull request before landing. |
@gitbw95 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@gitbw95 has updated the pull request. You must reimport the pull request before landing. |
@gitbw95 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Summary:
enable_custom_split_merge
is added for enabling the custom split and merge feature, which split the compressed value into chunks so that they may better fit jemalloc bins.Test Plan:
Unit Tests
Stress Tests