From 579586c6970f63e92e3ca1883f358ce506f14c4a Mon Sep 17 00:00:00 2001 From: Michael Stack Date: Fri, 19 Jan 2018 10:59:47 -0800 Subject: [PATCH] HBASE-19827 Addendum for Flakey TestAssignmentManager Don't clear old location from RTP... --- .../hadoop/hbase/master/procedure/ServerCrashProcedure.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/ServerCrashProcedure.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/ServerCrashProcedure.java index e9be05800dc..c18d72534ea 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/ServerCrashProcedure.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/ServerCrashProcedure.java @@ -195,9 +195,9 @@ implements ServerProcedureInterface { // Assign meta if still carrying it. Check again: region may be assigned because of RIT timeout final AssignmentManager am = env.getMasterServices().getAssignmentManager(); for (RegionInfo hri: am.getRegionStates().getServerRegionInfoSet(serverName)) { - if (!isDefaultMetaRegion(hri)) continue; - - am.offlineRegion(hri); + if (!isDefaultMetaRegion(hri)) { + continue; + } addChildProcedure(new RecoverMetaProcedure(serverName, this.shouldSplitWal)); } }