From 8ef1ae32e684f0fff1fd6c94639fc3473f0f03d6 Mon Sep 17 00:00:00 2001 From: bsglz <18031031@qq.com> Date: Mon, 4 Apr 2022 20:25:42 +0800 Subject: [PATCH] HBASE-26885 Addendum throw exception instead of return in TRSP to let the procedure retry (#4299) Signed-off-by: Duo Zhang (cherry picked from commit 7f9d1f2259da5851b532ae93aaea75126c706688) --- .../hbase/master/assignment/TransitRegionStateProcedure.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/TransitRegionStateProcedure.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/TransitRegionStateProcedure.java index d2001c62ad11..2adb19f0ace1 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/TransitRegionStateProcedure.java +++ b/hbase-server/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));