Junping Du commented on YARN-313:

Thanks [~jianhe] for the review! 
bq. we may rename refreshResources to RefreshNodesResources to be more clear?
+1. That sounds better.

bq. About the config format, how about a more concise way such as below, this 
can be tracked separately if it makes sense. 
Sounds good. Given this jira is pending for a very long time, I will file a 
separated JIRA to address this optimization work. Is this OK?

> 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