This is an automated email from the ASF dual-hosted git repository.

zhangduo pushed a commit to branch branch-2
in repository https://gitbox.apache.org/repos/asf/hbase.git

commit 8ef1ae32e684f0fff1fd6c94639fc3473f0f03d6
Author: bsglz <[email protected]>
AuthorDate: Mon Apr 4 20:25:42 2022 +0800

    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 7f9d1f2259da5851b532ae93aaea75126c706688)
---
 .../hadoop/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 d2001c62ad1..2adb19f0ace 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 @@ public class TransitRegionStateProcedure
       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));

Reply via email to