You are viewing a plain text version of this content. The canonical link for it is here.
Posted to mapreduce-issues@hadoop.apache.org by "Ravi Gummadi (JIRA)" <ji...@apache.org> on 2011/04/08 10:29:06 UTC

[jira] [Commented] (MAPREDUCE-2416) In Gridmix, in RoundRobinUserResolver, the list of groups for a user obtained from users-list-file is incorrect

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

Ravi Gummadi commented on MAPREDUCE-2416:
-----------------------------------------

Since Gridmix doesn't use/need the groups specified in users-list-file, I am planning to remove the restriction of specifying the groups in this file for RoundRobinUserResolver.

So the new format of the users-list-file expected is

user1
user2
user3
....


Just to allow backward compatibility, the list of groups can be there after user name after comma in any/all the lines.
So each line can be of the form

username[,group]*

> In Gridmix, in RoundRobinUserResolver, the list of groups for a user obtained from users-list-file is incorrect
> ---------------------------------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-2416
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-2416
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: contrib/gridmix
>            Reporter: Ravi Gummadi
>            Assignee: Ravi Gummadi
>
> RoundRobinUserResolver.parseUserList() has a bug in obtaining list of groups for each user --- in the sense that the list is not cleared before obtaining groups list for the next user. So if the first line has some groups, then from 2nd
> line onwards, the validation of "whether the users(in the next lines) are also having group names in those lines" is useless as the list is already nonempty.
> For example, users-list-file content as shown below also is valid as per parseUserList():
> ------------------
> user1,group1
> user2,
> user3,
> user4,
> ------------------

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira