From 459c7fba36fbcbfe848947bf3e3e903895b2233c Mon Sep 17 00:00:00 2001 From: sdong Date: Mon, 30 Nov 2015 21:32:59 -0800 Subject: [PATCH] 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 --- db/db_iter.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/db/db_iter.cc b/db/db_iter.cc index d04d30e9e..74558d56d 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