You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by zj...@apache.org on 2013/11/14 03:13:57 UTC

svn commit: r1541783 - /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java

Author: zjushch
Date: Thu Nov 14 02:13:57 2013
New Revision: 1541783

URL: http://svn.apache.org/r1541783
Log:
HBASE-9810 Global memstore size will be calculated wrongly if replaying recovered edits throws exception

Modified:
    hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java

Modified: hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
URL: http://svn.apache.org/viewvc/hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java?rev=1541783&r1=1541782&r2=1541783&view=diff
==============================================================================
--- hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java (original)
+++ hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java Thu Nov 14 02:13:57 2013
@@ -2851,11 +2851,11 @@ public class HRegion implements HeapSize
           throw e;
         }
       }
-      // The edits size added into rsAccounting during this replaying will not
-      // be required any more. So just clear it.
-      if (this.rsAccounting != null) {
-        this.rsAccounting.clearRegionReplayEditsSize(this.getRegionName());
-      }
+    }
+    // The edits size added into rsAccounting during this replaying will not
+    // be required any more. So just clear it.
+    if (this.rsAccounting != null) {
+      this.rsAccounting.clearRegionReplayEditsSize(this.getRegionName());
     }
     if (seqid > minSeqIdForTheRegion) {
       // Then we added some edits to memory. Flush and cleanup split edit files.