From: sdong <siying.d@fb.com>
Date: Tue, 1 Dec 2015 05:32:59 +0000 (-0800)
Subject: Revert previous behavior of internal_key_skipped_count
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=459c7fba36fbcbfe848947bf3e3e903895b2233c;p=rocksdb.git

Revert previous behavior of internal_key_skipped_count

Summary: With recent commit 33e0c93826fd45640b24324a642ca03d8b9ef123, db iterator skips perf context counter internal_key_skipped_count when blindly issuing internal Next(). Now increment the counter by one when issuing this Next()

Test Plan: Run all existing tests

Reviewers: rven, yhchiang, IslamAbdelRahman, kradhakrishnan, igor, anthony

Reviewed By: anthony

Subscribers: yoshinorim, leveldb, dhruba

Differential Revision: https://reviews.facebook.net/D51465
---

diff --git a/db/db_iter.cc b/db/db_iter.cc
index d04d30e9..74558d56 100644
--- a/db/db_iter.cc
+++ b/db/db_iter.cc
@@ -193,6 +193,7 @@ void DBIter::Next() {
     // If the current key is a merge, very likely iter already points
     // to the next internal position.
     iter_->Next();
+    PERF_COUNTER_ADD(internal_key_skipped_count, 1);
   }
 
   // Now we point to the next internal position, for both of merge and