-
Notifications
You must be signed in to change notification settings - Fork 3.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HBASE-24117 Shutdown AssignmentManager before ProcedureExecutor may c… #1865
Conversation
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some nits you might want to fix otherwise, lets try it. Do against branch-2 because there we'll notice if the test fails since it stable.
hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
Outdated
Show resolved
Hide resolved
@@ -485,6 +486,12 @@ private void assignRegions(MasterProcedureEnv env, List<RegionInfo> regions) thr | |||
// UPDATE: HBCKServerCrashProcedure overrides isMatchingRegionLocation; this check can get | |||
// in the way of our clearing out 'Unknown Servers'. | |||
if (!isMatchingRegionLocation(regionNode)) { | |||
// See HBASE-24117, though we have already changed the shutdown order, it is still worth | |||
// double checking here to confirm that we do not skip assignment incorrectly. | |||
if (!am.isRunning()) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good.
…ause SCP to accidentally skip assigning a region
🎊 +1 overall
This message was automatically generated. |
…ause SCP to accidentally skip assigning a region (#1865) Signed-off-by: Michael Stack <stack@apache.org>
…ause SCP to accidentally skip assigning a region (#1865) Signed-off-by: Michael Stack <stack@apache.org>
…ause SCP to accidentally skip assigning a region (#1865) Signed-off-by: Michael Stack <stack@apache.org>
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
…ause SCP to accidentally skip assigning a region (apache#1865) Signed-off-by: Michael Stack <stack@apache.org>
…ause SCP to accidentally skip assigning a region (apache#1865) Signed-off-by: Michael Stack <stack@apache.org>
…ause SCP to accidentally skip assigning a region (apache#1865) Signed-off-by: Michael Stack <stack@apache.org>
…ause SCP to accidentally skip assigning a region (apache#1865) Author: Duo Zhang Reason: Bug Ref: CDPD-15964 Signed-off-by: Michael Stack <stack@apache.org> Change-Id: I6bb4b16b68ea8c9e5baa39aa719447268da24c5c (cherry picked from commit 3419317)
…ause SCP to accidentally skip assigning a region (apache#1865) Signed-off-by: Michael Stack <stack@apache.org> (cherry picked from commit dee9933) Change-Id: I6bb4b16b68ea8c9e5baa39aa719447268da24c5c
…ause SCP to accidentally skip assigning a region