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

Vinod K V commented on MAPREDUCE-1754:
--------------------------------------

Looked at the patch. Mostly looks good.
 - It is still possible to provide backwards compatibility for 
{{mapreduce.cluster.permissions.supergroup}} by simply adding the deprecated 
group to the new ACL list. If we do this, we should rollback the changes to 
ConfigUtil.java and also add one simple test.
 - TestNodeRefresh: For the change in testMRRefreshDefault(), the comment 
should read as // refresh by invalid user.

> Replace mapred.persmissions.supergroup with an acl : 
> mapreduce.cluster.administrators
> -------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-1754
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-1754
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: jobtracker
>            Reporter: Amareshwari Sriramadasu
>            Assignee: Amareshwari Sriramadasu
>             Fix For: 0.22.0
>
>         Attachments: patch-1754-ydist.txt, patch-1754.txt
>
>
> mapred.permissions.supergroup should be replaced with an acl so that it does 
> not restrict the admins to a single group.
> See more details on MAPREDUCE-1542.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to