Skip to content
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 a memtable-only iterator option #1953

Closed
wants to merge 3 commits into from

Conversation

sagar0
Copy link
Contributor

@sagar0 sagar0 commented Mar 7, 2017

This PR is to support a way to iterate over all the keys that are just in memtables.

Test Plan:

  • Added a new unit-test
  • make check

@facebook-github-bot
Copy link
Contributor

@sagar0 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@sagar0 sagar0 changed the title Add a memtable-only iterator Add a memtable-only iterator option Mar 7, 2017
@sagar0 sagar0 requested a review from siying March 7, 2017 18:54
Copy link
Contributor

@siying siying left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

facebook-github-bot pushed a commit that referenced this pull request Nov 3, 2017
Summary:
This options was introduced in the C++ API in #1953 .
Closes #3064

Differential Revision: D6139010

Pulled By: sagar0

fbshipit-source-id: 164de11d539d174cf3afe7cd40e667049f44b0bc
benesch pushed a commit to benesch/rocksdb that referenced this pull request Nov 6, 2017
Summary:
This options was introduced in the C++ API in facebook#1953 .
Closes facebook#3064

Differential Revision: D6139010

Pulled By: sagar0

fbshipit-source-id: 164de11d539d174cf3afe7cd40e667049f44b0bc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants