You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by jg...@apache.org on 2009/07/29 02:58:40 UTC

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

Author: jgray
Date: Wed Jul 29 00:58:40 2009
New Revision: 798740

URL: http://svn.apache.org/viewvc?rev=798740&view=rev
Log:
HBASE-1718 Reuse of KeyValue during log replay could cause the wrong data to be used

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

Modified: hadoop/hbase/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/CHANGES.txt?rev=798740&r1=798739&r2=798740&view=diff
==============================================================================
--- hadoop/hbase/trunk/CHANGES.txt (original)
+++ hadoop/hbase/trunk/CHANGES.txt Wed Jul 29 00:58:40 2009
@@ -300,6 +300,8 @@
                using copies
    HBASE-1647  Filter#filterRow is called too often, filters rows it shouldn't
                have (Doğacan Güney via Ryan Rawson and Stack)
+   HBASE-1718  Reuse of KeyValue during log replay could cause the wrong
+               data to be used
 
   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/Store.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/Store.java?rev=798740&r1=798739&r2=798740&view=diff
==============================================================================
--- hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/Store.java (original)
+++ hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/Store.java Wed Jul 29 00:58:40 2009
@@ -329,6 +329,8 @@
         if (reporter != null && (editsCount % reportInterval) == 0) {
           reporter.progress();
         }
+        // Instantiate a new KeyValue to perform Writable on
+        val = new KeyValue();
       }
       if (LOG.isDebugEnabled()) {
         LOG.debug("Applied " + editsCount + ", skipped " + skippedEdits +