[
https://issues.apache.org/jira/browse/YARN-2500?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14167962#comment-14167962
]
Vinod Kumar Vavilapalli commented on YARN-2500:
-----------------------------------------------
This looks good. One nit:
{code}
if (null == amReq.getPriority()) {
amReq.setPriority(RMAppAttemptImpl.AM_CONTAINER_PRIORITY);
}
{code}
We should enforce it like the way we are enforcing num-containers to be 1.
> Miscellaneous changes in ResourceManager to support labels
> ----------------------------------------------------------
>
> Key: YARN-2500
> URL: https://issues.apache.org/jira/browse/YARN-2500
> Project: Hadoop YARN
> Issue Type: Sub-task
> Components: resourcemanager
> Reporter: Wangda Tan
> Assignee: Wangda Tan
> Attachments: YARN-2500-20141009-1.patch, YARN-2500-20141009-2.patch,
> YARN-2500.patch, YARN-2500.patch, YARN-2500.patch, YARN-2500.patch,
> YARN-2500.patch
>
>
> This patches contains changes in ResourceManager to support labels
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
