Use skiplist rep for range tombstone memtable

Summary: somehow missed committing this update in D62217

Test Plan: make check

Reviewers: sdong

Reviewed By: sdong

Subscribers: andrewkr, dhruba, leveldb

Differential Revision: https://reviews.facebook.net/D65361

diff --git a/db/db_properties_test.cc b/db/db_properties_test.cc
index 0ea1d7d92b11144b69185d807f05cff7511f9150..e0ad3ab9f2e369d85e0b1d0eb523121cccafc07e 100644 (file)
--- a/db/db_properties_test.cc
+++ b/db/db_properties_test.cc
@@ -558,7 +558,7 @@ TEST_F(DBPropertiesTest, NumImmutableMemTable) {
     ASSERT_TRUE(dbfull()->GetProperty(
         handles_[1], "rocksdb.cur-size-active-mem-table", &num));
     // "384" is the size of the metadata of two empty skiplists, this would
-    // break if we change the default vectorrep/skiplist implementation
+    // break if we change the default skiplist implementation
     ASSERT_EQ(num, "384");
 
     uint64_t int_num;

diff --git a/db/memtable.cc b/db/memtable.cc
index b3f0b65e1447a08daeaf6b1add389d00bb97c631..338dcf69e60b6c59efffa4d0fb4d01010a5a02f8 100644 (file)
--- a/db/memtable.cc
+++ b/db/memtable.cc
@@ -68,7 +68,7 @@ MemTable::MemTable(const InternalKeyComparator& cmp,
       table_(ioptions.memtable_factory->CreateMemTableRep(
           comparator_, &allocator_, ioptions.prefix_extractor,
           ioptions.info_log)),
-      range_del_table_(ioptions.memtable_factory->CreateMemTableRep(
+      range_del_table_(SkipListFactory().CreateMemTableRep(
           comparator_, &allocator_, nullptr /* transform */,
           ioptions.info_log)),
       data_size_(0),