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/04/30 00:27:44 UTC

svn commit: r769973 - in /hadoop/hbase/trunk: CHANGES.txt src/java/org/apache/hadoop/hbase/regionserver/Memcache.java

Author: stack
Date: Wed Apr 29 22:27:44 2009
New Revision: 769973

URL: http://svn.apache.org/viewvc?rev=769973&view=rev
Log:
HBASE-1358  Bug in reading from Memcache method (read only from snapshot)

Modified:
    hadoop/hbase/trunk/CHANGES.txt
    hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/Memcache.java

Modified: hadoop/hbase/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/CHANGES.txt?rev=769973&r1=769972&r2=769973&view=diff
==============================================================================
--- hadoop/hbase/trunk/CHANGES.txt (original)
+++ hadoop/hbase/trunk/CHANGES.txt Wed Apr 29 22:27:44 2009
@@ -92,6 +92,8 @@
    HBASE-1320  hbase-1234 broke filter tests
    HBASE-1355  [performance] Cache family maxversions; we were calculating on
                each access
+   HBASE-1358  Bug in reading from Memcache method (read only from snapshot)
+               (Evgeny Ryabitskiy via Stack)
 
   IMPROVEMENTS
    HBASE-1089  Add count of regions on filesystem to master UI; add percentage

Modified: hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/Memcache.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/Memcache.java?rev=769973&r1=769972&r2=769973&view=diff
==============================================================================
--- hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/Memcache.java (original)
+++ hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/Memcache.java Wed Apr 29 22:27:44 2009
@@ -578,6 +578,7 @@
       if (this.comparator.matchingRowColumn(kv, key)) {
         if (Store.doKeyValue(kv, versions, deletes, now, this.ttl, keyvalues,
             tailset)) {
+          enoughVersions = true;
           break;
         }
       } else {