infraio commented on issue #574: HBASE-22964 Fix flaky TestClusterRestartFailover and TestClusterResta… URL: https://github.com/apache/hbase/pull/574#issuecomment-527263907 > We need to know what is the purpose of the test. The current fix looks like we just give up testing if some conditions are not match... It was added by HBASE-21565: Delete dead server from dead server list too early leads to concurrent Server Crash Procedures(SCP) for a same server. So the test purpose is assertFalse(UTIL.getHBaseCluster().getMaster().getServerManager().expireServer(testServer)). It should not submit SCP again if there arleady had a SCP for this server.
---------------------------------------------------------------- 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
