[ https://issues.apache.org/jira/browse/YARN-6918?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16307723#comment-16307723 ]
Bibin A Chundatt edited comment on YARN-6918 at 1/2/18 6:37 AM: ---------------------------------------------------------------- Thank you [~sunilg] for review {quote} Why removePermission needs ugi as second param? {quote} Followed the same as {{setPermission}} the current implementation doesnt use it . Letting {{authorization-provider}} know who did remove call. Need to update patch and change to {{UserGroupInformation.getCurrentUser()}} instead of {{null}} in remove call. was (Author: bibinchundatt): Thank you [~sunilg] for review {quote} Why removePermission needs ugi as second param? {quote} Followed the same as {{setPermission}} the current implementation doesnt use it . Letting {{authorization-provider}} know who did remove call. > Remove acls after queue delete to avoid memory leak > --------------------------------------------------- > > Key: YARN-6918 > URL: https://issues.apache.org/jira/browse/YARN-6918 > Project: Hadoop YARN > Issue Type: Bug > Components: capacity scheduler > Reporter: Bibin A Chundatt > Assignee: Bibin A Chundatt > Attachments: YARN-6918.001.patch > > > Acl for deleted queue need to removed from allAcls to avoid leak > (Priority,YarnAuthorizer) -- This message was sent by Atlassian JIRA (v6.4.14#64029) --------------------------------------------------------------------- To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org