HBASE-26885 Addendum throw exception instead of return in TRSP to let…

… the procedure retry (#4299) Signed-off-by: Duo Zhang <[email protected]> (cherry picked from commit 7f9d1f2)
apache · Apr 4, 2022 · 8ef1ae3 · 8ef1ae3
1 parent 3b38380
commit 8ef1ae3
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/.../src/main/java/org/apache/hadoop/hbase/master/assignment/TransitRegionStateProcedure.java b/.../src/main/java/org/apache/hadoop/hbase/master/assignment/TransitRegionStateProcedure.java
@@ -205,7 +205,7 @@ private void openRegion(MasterProcedureEnv env, RegionStateNode regionNode) thro
       LOG.warn("No location specified for {}, jump back to state {} to get one", getRegion(),
         RegionStateTransitionState.REGION_STATE_TRANSITION_GET_ASSIGN_CANDIDATE);
       setNextState(RegionStateTransitionState.REGION_STATE_TRANSITION_GET_ASSIGN_CANDIDATE);
-      return;
+      throw new HBaseIOException("Failed to open region, the location is null or bogus.");
     }
     env.getAssignmentManager().regionOpening(regionNode);
     addChildProcedure(new OpenRegionProcedure(this, getRegion(), loc));