You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by la...@apache.org on 2012/08/15 19:55:34 UTC
svn commit: r1373540 -
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
Author: larsh
Date: Wed Aug 15 17:55:34 2012
New Revision: 1373540
URL: http://svn.apache.org/viewvc?rev=1373540&view=rev
Log:
HBASE-6579 Unnecessary KV order check in StoreScanner
Modified:
hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
Modified: hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java?rev=1373540&r1=1373539&r2=1373540&view=diff
==============================================================================
--- hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java (original)
+++ hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java Wed Aug 15 17:55:34 2012
@@ -342,11 +342,8 @@ public class StoreScanner extends NonLaz
LOOP: while((kv = this.heap.peek()) != null) {
// Check that the heap gives us KVs in an increasing order.
- if (prevKV != null && comparator != null
- && comparator.compare(prevKV, kv) > 0) {
- throw new IOException("Key " + prevKV + " followed by a " +
- "smaller key " + kv + " in cf " + store);
- }
+ assert prevKV == null || comparator == null || comparator.compare(prevKV, kv) <= 0 :
+ "Key " + prevKV + " followed by a " + "smaller key " + kv + " in cf " + store;
prevKV = kv;
ScanQueryMatcher.MatchCode qcode = matcher.match(kv);
switch(qcode) {