HBASE-26885 The TRSP should not go on when it get a bogus server name… (#4276)
* HBASE-26885 The TRSP should not go on when it get a bogus server name from AM
(cherry picked from commit 1efd8fe53c
)
This commit is contained in:
parent
543128db4a
commit
a25e779f26
|
@ -17,6 +17,8 @@
|
||||||
*/
|
*/
|
||||||
package org.apache.hadoop.hbase.master.assignment;
|
package org.apache.hadoop.hbase.master.assignment;
|
||||||
|
|
||||||
|
import static org.apache.hadoop.hbase.master.LoadBalancer.BOGUS_SERVER_NAME;
|
||||||
|
|
||||||
import edu.umd.cs.findbugs.annotations.Nullable;
|
import edu.umd.cs.findbugs.annotations.Nullable;
|
||||||
import java.io.IOException;
|
import java.io.IOException;
|
||||||
import org.apache.hadoop.hbase.HBaseIOException;
|
import org.apache.hadoop.hbase.HBaseIOException;
|
||||||
|
@ -199,7 +201,7 @@ public class TransitRegionStateProcedure
|
||||||
|
|
||||||
private void openRegion(MasterProcedureEnv env, RegionStateNode regionNode) throws IOException {
|
private void openRegion(MasterProcedureEnv env, RegionStateNode regionNode) throws IOException {
|
||||||
ServerName loc = regionNode.getRegionLocation();
|
ServerName loc = regionNode.getRegionLocation();
|
||||||
if (loc == null) {
|
if (loc == null || BOGUS_SERVER_NAME.equals(loc)) {
|
||||||
LOG.warn("No location specified for {}, jump back to state {} to get one", getRegion(),
|
LOG.warn("No location specified for {}, jump back to state {} to get one", getRegion(),
|
||||||
RegionStateTransitionState.REGION_STATE_TRANSITION_GET_ASSIGN_CANDIDATE);
|
RegionStateTransitionState.REGION_STATE_TRANSITION_GET_ASSIGN_CANDIDATE);
|
||||||
setNextState(RegionStateTransitionState.REGION_STATE_TRANSITION_GET_ASSIGN_CANDIDATE);
|
setNextState(RegionStateTransitionState.REGION_STATE_TRANSITION_GET_ASSIGN_CANDIDATE);
|
||||||
|
|
Loading…
Reference in New Issue