You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2009/10/19 22:09:09 UTC

svn commit: r826793 - in /hadoop/hbase/branches/0.20: CHANGES.txt src/java/org/apache/hadoop/hbase/regionserver/HRegion.java

Author: stack
Date: Mon Oct 19 20:09:08 2009
New Revision: 826793

URL: http://svn.apache.org/viewvc?rev=826793&view=rev
Log:
HBASE-1917 TestScanner.testFilters failing

Modified:
    hadoop/hbase/branches/0.20/CHANGES.txt
    hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/regionserver/HRegion.java

Modified: hadoop/hbase/branches/0.20/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.20/CHANGES.txt?rev=826793&r1=826792&r2=826793&view=diff
==============================================================================
--- hadoop/hbase/branches/0.20/CHANGES.txt (original)
+++ hadoop/hbase/branches/0.20/CHANGES.txt Mon Oct 19 20:09:08 2009
@@ -11,6 +11,7 @@
    HBASE-1912  When adding a secondary index to an existing table, it will
                cause NPE during re-indexing (Mingjui Ray Liao via Andrew
                Purtell)
+   HBASE-1917  TestScanner.testFilters failing
 
   IMPROVEMENTS
    HBASE-1899  Use scanner caching in shell count

Modified: hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/regionserver/HRegion.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/regionserver/HRegion.java?rev=826793&r1=826792&r2=826793&view=diff
==============================================================================
--- hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/regionserver/HRegion.java (original)
+++ hadoop/hbase/branches/0.20/src/java/org/apache/hadoop/hbase/regionserver/HRegion.java Mon Oct 19 20:09:08 2009
@@ -1765,7 +1765,8 @@
           if (hasResults()) return true;
         }
         // See if current row should be filtered based on row key
-        if (this.filter != null && this.filter.filterRowKey(row, 0, row.length)) {
+        if ((this.filter != null && this.filter.filterRowKey(row, 0, row.length)) ||
+            (oldFilter != null && this.oldFilter.filterRowKey(row, 0, row.length))) {
           readAndDumpCurrentResult();
           resetFilters();
           filterCurrentRow = true;