svn commit: r1509800 - /hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/master/ThrottledRegionReopener.java

[li...@apache.org]

Author: liyin
Date: Fri Aug  2 18:18:18 2013
New Revision: 1509800

URL: http://svn.apache.org/r1509800
Log:
[HBASE-9114] Fix TestChangingEncoding

Author: manukranthk

Summary: TestChangingEncoding tries to do alterTable when the table is disabled. The region reopener is not activated if the table is disabled, which seems unintuitive. So, removing the check to fix this test.

Test Plan: Run mrunit and check the failed tests locally

Reviewers: liyintang, aaiyer

Reviewed By: liyintang

CC: hbase-eng@

Differential Revision: https://phabricator.fb.com/D911664

Task ID: 2656657

Modified:
    hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/master/ThrottledRegionReopener.java

Modified: hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/master/ThrottledRegionReopener.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/master/ThrottledRegionReopener.java?rev=1509800&r1=1509799&r2=1509800&view=diff
==============================================================================
--- hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/master/ThrottledRegionReopener.java (original)
+++ hbase/branches/0.89-fb/src/main/java/org/apache/hadoop/hbase/master/ThrottledRegionReopener.java Fri Aug  2 18:18:18 2013
@@ -181,10 +181,8 @@ public class ThrottledRegionReopener {
    * @throws IOException
    */
   public void startRegionsReopening() throws IOException {
-    if (HTable.isTableEnabled(master.getConfiguration(), tableName)) {
-      LOG.info("Initiating reopen for all regions of " + tableName);
-      closer.start();
-    }
+    LOG.info("Initiating reopen for all regions of " + tableName);
+    closer.start();
   }
 
   public int getRegionCloseWaitInterval() {