You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by te...@apache.org on 2012/05/17 18:27:16 UTC

svn commit: r1339688 - /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java

Author: tedyu
Date: Thu May 17 16:27:16 2012
New Revision: 1339688

URL: http://svn.apache.org/viewvc?rev=1339688&view=rev
Log:
HBASE-6029 HBCK doesn't recover Balance switch if exception occurs in onlineHbck() (Maryann)

Modified:
    hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java

Modified: hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java?rev=1339688&r1=1339687&r2=1339688&view=diff
==============================================================================
--- hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java (original)
+++ hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java Thu May 17 16:27:16 2012
@@ -378,10 +378,12 @@ public class HBaseFsck {
 
     // turn the balancer off
     boolean oldBalancer = admin.balanceSwitch(false);
-
-    onlineConsistencyRepair();
-
-    admin.balanceSwitch(oldBalancer);
+    try {
+      onlineConsistencyRepair();
+    }
+    finally {
+      admin.balanceSwitch(oldBalancer);
+    }
 
     // Print table summary
     printTableSummary(tablesInfo);