[
https://issues.apache.org/jira/browse/YARN-41?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14569182#comment-14569182
]
Junping Du commented on YARN-41:
--------------------------------
Thanks [~devaraj.k] for updating the patch with addressing previous comments!
Latest patch LGTM. +1. Will commit it tomorrow if no further comments on the
code from other reviewers.
In addition, given the patch involve new SHUTDOWN category on: NodeState, UI
and Cluster Metrics. Although it doesn't break any public APIs, we should mark
this JIRA as incompatible for its inconsistent behaviors with previous releases
in UI, CLI, Metrics (to notify users or third-party management & monitor
software). In general, I think it should be fine to keep the plan to include
this patch in 2.x releases. However, please comments here to let us know if you
have any concerns.
> The RM should handle the graceful shutdown of the NM.
> -----------------------------------------------------
>
> Key: YARN-41
> URL: https://issues.apache.org/jira/browse/YARN-41
> Project: Hadoop YARN
> Issue Type: New Feature
> Components: nodemanager, resourcemanager
> Reporter: Ravi Teja Ch N V
> Assignee: Devaraj K
> Attachments: MAPREDUCE-3494.1.patch, MAPREDUCE-3494.2.patch,
> MAPREDUCE-3494.patch, YARN-41-1.patch, YARN-41-2.patch, YARN-41-3.patch,
> YARN-41-4.patch, YARN-41-5.patch, YARN-41-6.patch, YARN-41-7.patch,
> YARN-41-8.patch, YARN-41.patch
>
>
> Instead of waiting for the NM expiry, RM should remove and handle the NM,
> which is shutdown gracefully.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
