-
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
Use GetBlobFileSize instead of GetTotalBlobBytes in DB properties #8902
Conversation
@ltamasi has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
@ltamasi has updated the pull request. You must reimport the pull request before landing. |
@ltamasi has imported this pull request. If you are a Facebook 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, maybe it's worth mentioning in the HISTORY?
We've just added these properties (they weren't part of any previous release), so to me it doesn't seem necessary in this specific case. |
…902) Summary: The patch adjusts the definition of BlobDB's DB properties a bit by switching to `GetBlobFileSize` from `GetTotalBlobBytes`. The difference is that the value returned by `GetBlobFileSize` includes the blob file header and footer as well, and thus matches the on-disk size of blob files. In addition, the patch removes the `Version` number from the `blob_stats` property, and updates/extends the unit tests a little. Pull Request resolved: facebook/rocksdb#8902 Test Plan: `make check` Reviewed By: riversand963 Differential Revision: D30859542 Pulled By: ltamasi fbshipit-source-id: e3426d2d567bd1bd8c8636abdafaafa0743c854c
Summary:
The patch adjusts the definition of BlobDB's DB properties a bit by
switching to
GetBlobFileSize
fromGetTotalBlobBytes
. Thedifference is that the value returned by
GetBlobFileSize
includesthe blob file header and footer as well, and thus matches the on-disk
size of blob files. In addition, the patch removes the
Version
numberfrom the
blob_stats
property, and updates/extends the unit tests a little.Test Plan:
make check