You are viewing a plain text version of this content. The canonical link for it is here.

Posted to yarn-issues@hadoop.apache.org by "Xuan Gong (JIRA)" <ji...@apache.org> on 2015/05/02 00:03:08 UTC

[jira] [Commented] (YARN-1949) Add admin ACL check to AdminService#updateNodeResource()

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

Xuan Gong commented on YARN-1949:
---------------------------------

[~kj-ki] Could you update the patch , please ? Given junping's comment "Looks like some other patch already include the ACL check in AdminService. However, the test code here still helpful"

> Add admin ACL check to AdminService#updateNodeResource()
> --------------------------------------------------------
>
>                 Key: YARN-1949
>                 URL: https://issues.apache.org/jira/browse/YARN-1949
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: resourcemanager
>            Reporter: Kenji Kikushima
>            Assignee: Kenji Kikushima
>         Attachments: YARN-1949.patch
>
>
> At present, updateNodeResource() doesn't check ACL. We should call checkAcls() before setResourceOption().



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)