From: Abhishek Kona <abhishekk@fb.com>
Date: Tue, 8 Jan 2013 23:55:06 +0000 (-0800)
Subject: Remove unnecessary asserts in table/merger.cc
X-Git-Tag: v1.5.7~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=2e1ad2c48b60aafe9a030f206456321ee04bb030;p=rocksdb.git

Remove unnecessary asserts in table/merger.cc

Summary:
The asserts introduced in https://reviews.facebook.net/D7629 are
wrong.

The direction of iteration is changed after the function call so they
assert's fail.

Test Plan: make clean check

Reviewers: dhruba

Reviewed By: dhruba

CC: leveldb

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

diff --git a/table/merger.cc b/table/merger.cc
index 5f7b24bd8..7c5ef92f4 100644
--- a/table/merger.cc
+++ b/table/merger.cc
@@ -188,7 +188,6 @@ class MergingIterator : public Iterator {
 };
 
 void MergingIterator::FindSmallest() {
-  assert (direction_ == kForward);
   if (minHeap_.empty()) {
     current_ = NULL;
   } else {
@@ -199,7 +198,6 @@ void MergingIterator::FindSmallest() {
 }
 
 void MergingIterator::FindLargest() {
-  assert(direction_ == kReverse);
   if (maxHeap_.empty()) {
     current_ = NULL;
   } else {