skip WriteBatch prep for empty transaction

ony · Oct 23, 2014 · 21efb28 · 21efb28
1 parent 88c41c6
commit 21efb28
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 0 deletions.
diff --git a/include/leveldb/memory_db.hpp b/include/leveldb/memory_db.hpp
@@ -46,6 +46,7 @@ namespace leveldb
         using map::size;
         using map::begin;
         using map::end;
+        using map::empty;
 
         Status Get(const Slice &key, std::string &value) noexcept override
         {

diff --git a/include/leveldb/txn_db.hpp b/include/leveldb/txn_db.hpp
@@ -133,6 +133,8 @@ namespace leveldb
 
         Status commit()
         {
+            if (whiteout.empty() || overlay.empty()) return Status::OK();
+
             WriteBatch batch;
             for (auto k : whiteout) batch.Delete(k);
             for (auto kv : overlay) batch.Put(kv.first, kv.second);

diff --git a/include/leveldb/whiteout_db.hpp b/include/leveldb/whiteout_db.hpp
@@ -41,6 +41,7 @@ namespace leveldb
 
         using set::begin;
         using set::end;
+        using set::empty;
 
         bool Check(const Slice &key)
         { return find(key.ToString()) != end(); }