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 "Junping Du (JIRA)" <ji...@apache.org> on 2014/10/08 13:02:33 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=14163329#comment-14163329 ] 

Junping Du commented on YARN-1949:
----------------------------------

Sorry for coming here a little late... Looks like some other patch already include the ACL check in AdminService. However, the test code here still helpful. [~kj-ki], would you like to sync up the patch? I will help to review it. Thx!

> 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)