[ https://issues.apache.org/jira/browse/HBASE-3282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12964891#action_12964891 ]
HBase Review Board commented on HBASE-3282: ------------------------------------------- Message from: st...@duboce.net ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://review.cloudera.org/r/1259/#review2004 ----------------------------------------------------------- Ship it! branches/0.90/src/main/java/org/apache/hadoop/hbase/master/ServerManager.java <http://review.cloudera.org/r/1259/#comment6320> You can make this private now that its no longer referenced by Master? - stack > Need to retain DeadServers to ensure we don't allow previously expired RS > instances to rejoin cluster > ----------------------------------------------------------------------------------------------------- > > Key: HBASE-3282 > URL: https://issues.apache.org/jira/browse/HBASE-3282 > Project: HBase > Issue Type: Bug > Components: master > Affects Versions: 0.90.0 > Reporter: Jonathan Gray > Assignee: Jonathan Gray > Fix For: 0.90.0, 0.92.0 > > > Currently we clear a server from the deadserver set once we finish processing > it's shutdown. However, certain circumstances (network partitions, race > conditions) could lead to the RS not doing a check-in until after the > shutdown has been processed. As-is, this RS will now be let back in to the > cluster rather than rejected with YouAreDeadException. > We should hang on to the dead servers so we always reject them. > One concern is that the set will grow indefinitely. One recommendation by > stack is to use SoftReferences. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.