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

kv/KeyValueDB: add column family #18049

Merged
merged 11 commits into from
Oct 6, 2017
Merged

Commits on Oct 3, 2017

  1. os/ObjectMap: tie ObjectMap::ObjectMapIterator to superclass

    These iterfaces are similar but ultimately unrelated! Unfortunately
    we are stuck with a joined class hiearchy because of DBObjectMap.
    Someday it would be nice to break this link.
    
    Signed-off-by: Sage Weil <sage@redhat.com>
    liewegas committed Oct 3, 2017
    Configuration menu
    Copy the full SHA
    c9d54b6 View commit details
    Browse the repository at this point in the history
  2. kv/KeyValueDB: use public iterator interface simple; hide prefixing one

    This uses the most general class as the type for the Iterator
    shared_ptr typedef, and makes the class that filters by prefix
    private.
    
    Signed-off-by: Sage Weil <sage@redhat.com>
    liewegas committed Oct 3, 2017
    Configuration menu
    Copy the full SHA
    953232c View commit details
    Browse the repository at this point in the history
  3. kv/KeyValueDB: simplify iterator overloads

    get_wholespace_iterator() and get_iterator().  This gets rid of
    the goofy _get_iterator() indirection and avoids colliding on names w/
    virtual methods.
    
    Signed-off-by: Sage Weil <sage@redhat.com>
    liewegas committed Oct 3, 2017
    Configuration menu
    Copy the full SHA
    24b83ae View commit details
    Browse the repository at this point in the history
  4. kv/KineticStore: override

    Signed-off-by: Sage Weil <sage@redhat.com>
    liewegas committed Oct 3, 2017
    Configuration menu
    Copy the full SHA
    ba3353a View commit details
    Browse the repository at this point in the history
  5. kv/KeyValueDB: add column family interface

    Signed-off-by: Jianjian Huo <jianjian.huo@ssi.samsung.com>
    Signed-off-by: Sage Weil <sage@redhat.com>
    Jianjian Huo authored and liewegas committed Oct 3, 2017
    Configuration menu
    Copy the full SHA
    6bddacb View commit details
    Browse the repository at this point in the history
  6. kv/RocksDBStore: move to_bufferlist to .cc

    Signed-off-by: Sage Weil <sage@redhat.com>
    liewegas committed Oct 3, 2017
    Configuration menu
    Copy the full SHA
    ddc4ff9 View commit details
    Browse the repository at this point in the history

Commits on Oct 5, 2017

  1. kv/RocksDBStore: add CF support

    - detect and use existing CFs on open
    
    Signed-off-by: Jianjian Huo <jianjian.huo@ssi.samsung.com>
    Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
    Signed-off-by: Sage Weil <sage@redhat.com>
    Jianjian Huo authored and liewegas committed Oct 5, 2017
    Configuration menu
    Copy the full SHA
    1336028 View commit details
    Browse the repository at this point in the history
  2. kv/RocksDBStore: simplify CF merge operator routing

    Merge operators always map to a prefix, just like a CF, so we can link
    directly to the operator without doing any lookup.
    
    Signed-off-by: Sage Weil <sage@redhat.com>
    liewegas committed Oct 5, 2017
    Configuration menu
    Copy the full SHA
    895952f View commit details
    Browse the repository at this point in the history
  3. os/bluestore: define PREFIX_ALLOC_BITMAP

    This was hard-coded to 'b'.
    
    Signed-off-by: Sage Weil <sage@redhat.com>
    liewegas committed Oct 5, 2017
    Configuration menu
    Copy the full SHA
    87c1f57 View commit details
    Browse the repository at this point in the history
  4. os/bluestore: use separate CF for omap and deferred

    Signed-off-by: Sage Weil <sage@redhat.com>
    liewegas committed Oct 5, 2017
    Configuration menu
    Copy the full SHA
    6ea96fa View commit details
    Browse the repository at this point in the history
  5. kv/RocksDBStore: fix default CF handling

    We need to close the default CF handle *only* if we explicitly opened it.
    
    Also, take care to use that handle wherever we can.  This is mostly
    paranoia--I'm not sure if rocksdb cares if you mix the handle you opened
    with DefaultColumnFamily(), which is different.
    
    Signed-off-by: Sage Weil <sage@redhat.com>
    liewegas committed Oct 5, 2017
    Configuration menu
    Copy the full SHA
    f4e4323 View commit details
    Browse the repository at this point in the history