Apache9 commented on a change in pull request #852: HBASE-23322 [hbck2] 
Simplification on HBCKSCP scheduling
URL: https://github.com/apache/hbase/pull/852#discussion_r348251347
 
 

 ##########
 File path: 
hbase-server/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java
 ##########
 @@ -582,10 +562,7 @@ synchronized long expireServer(final ServerName 
serverName,
       return Procedure.NO_PROC_ID;
     }
     LOG.info("Processing expiration of " + serverName + " on " + 
this.master.getServerName());
-    long pid = function.apply(serverName);
-    if (pid <= 0) {
 
 Review comment:
   We do not need this test any more? It will skip the later listener calls.

----------------------------------------------------------------
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]


With regards,
Apache Git Services

Reply via email to