Jian He commented on YARN-313:

looks good overall, few comments,
we may rename refreshResources to RefreshNodesResources to be more clear ? this 
About the config format, how about a more concise way such as below, this can 
be tracked separately if it makes sense.
<NodeResource NodeId=localhost>
   <vcore> 1 </vcore>
   <memory> 1024 </memory>

> Add Admin API for supporting node resource configuration in command line
> ------------------------------------------------------------------------
>                 Key: YARN-313
>                 URL: https://issues.apache.org/jira/browse/YARN-313
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: client
>            Reporter: Junping Du
>            Assignee: Inigo Goiri
>            Priority: Critical
>         Attachments: YARN-313-sample.patch, YARN-313-v1.patch, 
> YARN-313-v10.patch, YARN-313-v2.patch, YARN-313-v3.patch, YARN-313-v4.patch, 
> YARN-313-v5.patch, YARN-313-v6.patch, YARN-313-v7.patch, YARN-313-v8.patch, 
> YARN-313-v9.patch
> We should provide some admin interface, e.g. "yarn rmadmin -refreshResources" 
> to support changes of node's resource specified in a config file.

This message was sent by Atlassian JIRA

Reply via email to