svn commit: r752537 - in /hadoop/hbase/branches/0.19: CHANGES.txt src/java/org/apache/hadoop/hbase/regionserver/StoreFileScanner.java

[stack]

Author: stack
Date: Wed Mar 11 17:27:17 2009
New Revision: 752537

URL: http://svn.apache.org/viewvc?rev=752537&view=rev
Log:
HBASE-1256  NPE in StoreFileScanner durring cache flush

Modified:
    hadoop/hbase/branches/0.19/CHANGES.txt
    
hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/regionserver/StoreFileScanner.java

Modified: hadoop/hbase/branches/0.19/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/hbase/branches/0.19/CHANGES.txt?rev=752537&r1=752536&r2=752537&view=diff
==============================================================================
--- hadoop/hbase/branches/0.19/CHANGES.txt (original)
+++ hadoop/hbase/branches/0.19/CHANGES.txt Wed Mar 11 17:27:17 2009
@@ -27,6 +27,7 @@
    HBASE-1219  Scanners can miss values riding the flush transition
    HBASE-1220  Don't reopen file if already open when updating readers
                underneath scanners
+   HBASE-1256  NPE in StoreFileScanner durring cache flush
 
 
   IMPROVEMENTS

Modified: 
hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/regionserver/StoreFileScanner.java
URL: 
http://svn.apache.org/viewvc/hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/regionserver/StoreFileScanner.java?rev=752537&r1=752536&r2=752537&view=diff
==============================================================================
--- 
hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/regionserver/StoreFileScanner.java
 (original)
+++ 
hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/regionserver/StoreFileScanner.java
 Wed Mar 11 17:27:17 2009
@@ -93,7 +93,7 @@
       } else {
         // Only add in non-null Readers
         int nonulls = 0;
-        for (int i = 1; i < this.readers.length; i++) {
+        for (int i = 0; i < this.readers.length; i++) {
           if (this.readers[i] != null) nonulls++;
         }
         MapFile.Reader [] newReaders = new MapFile.Reader[nonulls + 1];