[
https://issues.apache.org/jira/browse/HDFS-2920?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13218320#comment-13218320
]
Eli Collins commented on HDFS-2920:
-----------------------------------
Hey ATM, how about putting the NN health check implementation and delayed
shutdown to separate jiras? They seem like discrete, self-contained changes
(and things we can handle post merge if we want, though I think we can iterate
on them quickly today). Only have a couple comments on these parts, everything
else is good to go.
- !nameNodeHasResourcesAvailable implies "The NameNode has run out of
resources" instead of the
"NameNode is low on resources". It would be even better if the message was more
specific (eg mentioned lack of inodes of disk space).
- Wrt delayed shutdown, we likely have (or should have) similar code elsewhere
right since there's nothing HA specific? Why is the shutdown delayed rather
than immediate?
- Wrt "Error encountered during state transition.", isn't the error most likely
due to a failure to start a service?
> HA: fix remaining TODO items
> ----------------------------
>
> Key: HDFS-2920
> URL: https://issues.apache.org/jira/browse/HDFS-2920
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: ha
> Reporter: Eli Collins
> Assignee: Aaron T. Myers
> Attachments: HDFS-2920-HDFS-1623.patch, hdfs-2920-v1.txt
>
>
> There are a number of "TODO(HA)" and "TODO:HA" comments we need to fix or
> remove.
--
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