Apache9 commented on a change in pull request #2728:
URL: https://github.com/apache/hbase/pull/2728#discussion_r537055988
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java
##########
@@ -501,7 +501,8 @@ public DeadServer getDeadServers() {
* @return true if any RS are being processed as dead, false if not
*/
public boolean areDeadServersInProgress() throws IOException {
- return master.getProcedures().stream().anyMatch(p -> p instanceof
ServerCrashProcedure);
+ return
master.getMasterProcedureExecutor().getActiveProceduresNoCopy().stream()
Review comment:
The getActiveProceduresNoCopy is only expected to be called when
initialization, where we can make sure there is no race.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
