-
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
Organizing rocksdb/table directory by format #5373
Conversation
TARGETS
Outdated
"table/blockbased/flush_block_policy.cc", | ||
"table/blockbased/full_filter_block.cc", | ||
"table/blockbased/index_builder.cc", | ||
"table/blockbased/partitioned_filter_block.cc", |
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 don't have opinion on "blockbased" or "block_based" by itself, but since we already call "block_based" in file name, can we keep it consistent?
needs to fix cmake. |
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.
@siying 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.
@vjnadimpalli has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
@vjnadimpalli merged this pull request in 50e4707. |
Summary: Pull Request resolved: facebook#5373 Differential Revision: D15559425 Pulled By: vjnadimpalli fbshipit-source-id: 5d6d6d615582bedd96a4b879bb25d429a6de8b55
No description provided.