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

Reply via email to