Naganarasimha G R updated YARN-2923:
    Attachment: YARN-2923.20150517-1.patch

Hi [~wangda]
   Attaching a WIP patch (may be need some more static checks after jenkins 
# rebased the patch on top of 3565
# Moved common code which was earlier in 2729 to here as this jira will be 
going first
# corrected most of [~vinodkv]'s comments in 2729 and pending are :

* I think the format expected from the command should be more structured. 
Specifically as we expect more per-label attributes in line with YARN-3565.
* Not caused by your patch but worth fixing here. NodeStatusUpdaterImpl 
shouldn't worry about invalid label-set, previous-valid-labels and label 
validation. You should move all that functionality into NodeLabelsProvider.
* Can you add the documentation for setting this up too too?

For these wanted to discuss with you before working on it.

> Support configuration based NodeLabelsProvider Service in Distributed Node 
> Label Configuration Setup 
> -----------------------------------------------------------------------------------------------------
>                 Key: YARN-2923
>                 URL: https://issues.apache.org/jira/browse/YARN-2923
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: nodemanager
>            Reporter: Naganarasimha G R
>            Assignee: Naganarasimha G R
>             Fix For: 2.8.0
>         Attachments: YARN-2923.20141204-1.patch, YARN-2923.20141210-1.patch, 
> YARN-2923.20150328-1.patch, YARN-2923.20150404-1.patch, 
> YARN-2923.20150517-1.patch
> As part of Distributed Node Labels configuration we need to support Node 
> labels to be configured in Yarn-site.xml. And on modification of Node Labels 
> configuration in yarn-site.xml, NM should be able to get modified Node labels 
> from this NodeLabelsprovider service without NM restart

This message was sent by Atlassian JIRA

Reply via email to