You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by zh...@apache.org on 2022/04/04 12:25:48 UTC

[hbase] branch master updated: HBASE-26885 Addendum throw exception instead of return in TRSP to let the procedure retry (#4299)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 7f9d1f2259d HBASE-26885 Addendum throw exception instead of return in TRSP to let the procedure retry (#4299)
7f9d1f2259d is described below

commit 7f9d1f2259da5851b532ae93aaea75126c706688
Author: bsglz <18...@qq.com>
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 <zh...@apache.org>
---
 .../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));