hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From st...@apache.org
Subject svn commit: r1337413 - in /hbase/branches/0.90: CHANGES.txt src/main/java/org/apache/hadoop/hbase/io/HalfStoreFileReader.java
Date Fri, 11 May 2012 23:32:59 GMT
Author: stack
Date: Fri May 11 23:32:58 2012
New Revision: 1337413

URL: http://svn.apache.org/viewvc?rev=1337413&view=rev
Log:
HBASE-5922 HalfStoreFileReader seekBefore causes StackOverflowError

Modified:
    hbase/branches/0.90/CHANGES.txt
    hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/io/HalfStoreFileReader.java

Modified: hbase/branches/0.90/CHANGES.txt
URL: http://svn.apache.org/viewvc/hbase/branches/0.90/CHANGES.txt?rev=1337413&r1=1337412&r2=1337413&view=diff
==============================================================================
--- hbase/branches/0.90/CHANGES.txt (original)
+++ hbase/branches/0.90/CHANGES.txt Fri May 11 23:32:58 2012
@@ -29,6 +29,7 @@ Release 0.90.7 - Unreleased
    HBASE-5911  hbck need an option -help to printout usage
                (Jimmy Xiang)
    HBASE-5894  Table deletion failed but HBaseAdmin#deletetable reports it as success (Xufeng)
+   HBASE-5922  HalfStoreFileReader seekBefore causes StackOverflowError (Nate Putnam)
 
   IMPROVEMENT
    HBASE-5588  Deprecate/remove AssignmentManager#clearRegionFromTransition

Modified: hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/io/HalfStoreFileReader.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/io/HalfStoreFileReader.java?rev=1337413&r1=1337412&r2=1337413&view=diff
==============================================================================
--- hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/io/HalfStoreFileReader.java
(original)
+++ hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/io/HalfStoreFileReader.java
Fri May 11 23:32:58 2012
@@ -144,9 +144,10 @@ public class HalfStoreFileReader extends
             return false;
           }
         } else {
+          // The equals sign isn't strictly necessary just here to be consistent with seekTo
           if (getComparator().compare(key, offset, length, splitkey, 0,
               splitkey.length) >= 0) {
-            return seekBefore(splitkey, 0, splitkey.length);
+            return this.delegate.seekBefore(splitkey, 0, splitkey.length);
           }
         }
         return this.delegate.seekBefore(key, offset, length);



Mime
View raw message