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