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 2018/10/08 15:54:28 UTC

hbase git commit: HBASE-21271 [amv2] Don't throw UnsupportedOperationException when rollback called on Assign/Unassign; spiral of death

Repository: hbase
Updated Branches:
  refs/heads/branch-2.0 7d2154569 -> 73d67433a


HBASE-21271 [amv2] Don't throw UnsupportedOperationException when rollback called on Assign/Unassign; spiral of death


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/73d67433
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/73d67433
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/73d67433

Branch: refs/heads/branch-2.0
Commit: 73d67433af17d089ae8033e57e8f9354503815c7
Parents: 7d21545
Author: Michael Stack <st...@apache.org>
Authored: Thu Oct 4 16:03:39 2018 -0700
Committer: Michael Stack <st...@apache.org>
Committed: Tue Oct 9 00:54:14 2018 +0900

----------------------------------------------------------------------
 .../hbase/master/assignment/RegionTransitionProcedure.java     | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/73d67433/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/RegionTransitionProcedure.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/RegionTransitionProcedure.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/RegionTransitionProcedure.java
index 8f69698..9c87ab4 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/RegionTransitionProcedure.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/RegionTransitionProcedure.java
@@ -401,9 +401,9 @@ public abstract class RegionTransitionProcedure
 
     // There is no rollback for assignment unless we cancel the operation by
     // dropping/disabling the table.
-    throw new UnsupportedOperationException("Unhandled state " + transitionState +
-        "; there is no rollback for assignment unless we cancel the operation by " +
-        "dropping/disabling the table");
+    LOG.warn("Unhandled state {}; no rollback for assignment! Doing NOTHING!" +
+        " May need manual intervention. TODO: IS THIS WORKING? {}",
+        transitionState, this);
   }
 
   protected abstract boolean isRollbackSupported(final RegionTransitionState state);