BPOfferService#isAlive is poorly named --------------------------------------
Key: HDFS-2633 URL: https://issues.apache.org/jira/browse/HDFS-2633 Project: Hadoop HDFS Issue Type: Improvement Components: data-node Affects Versions: 0.23.0 Reporter: Eli Collins Priority: Minor Per HDFS-2627 the current implementation returns true even if one of the actor threads is dead. "The only non-test use case for isAlive seems to be from BlockPoolSliceScanner and DataBlockScanner, where they're really trying to figure out whether they should stop scanning the block pool. If the BPOS is connected to any NN at all (regardless of active/standby) it needs to report true so that the scanners don't stop running. It would be nice to clean up these calls and specify in their function name that they're only meant for use in tests" and annotate @VisibleForTesting. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira