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

Szilard Nemeth commented on YARN-10415:
---------------------------------------

Hi [~shuzirra],
I only have one comment: 
Typo in method name CSMappingPlacementRule#setupGroupsForVaribleContext
I can also agree to Peter's 2nd comment.

> Create a group matcher which checks ALL groups of the user
> ----------------------------------------------------------
>
>                 Key: YARN-10415
>                 URL: https://issues.apache.org/jira/browse/YARN-10415
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Gergely Pollak
>            Assignee: Gergely Pollak
>            Priority: Major
>         Attachments: YARN-10415.001.patch, YARN-10415.002.patch
>
>
> The current group matching is limited to %primary_group, which is a single 
> group. However the current behaviour of UserGroupMapping is we checks against 
> all of the groups of the user, and if it matches any of those, it will be a 
> match.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to