[jira] [Comment Edited] (YARN-313) Add Admin API for supporting node resource configuration in command line

["Jian He (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/YARN-313?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14740889#comment-14740889 ] 

Jian He edited comment on YARN-313 at 9/11/15 2:20 PM:
-------------------------------------------------------

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


was (Author: jianhe):
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.
{code}
<NodeResource NodeId=localhost>
   <vcore> 1 </vcore>
   <memory> 1024 </memory>
</NodeResource>
{code}

> 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
(v6.3.4#6332)