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

Siddharth Seth commented on MAPREDUCE-3903:
-------------------------------------------

Looked at the patch. Looks good. One minor nit - the debug log should be within 
an if debug enabled check.
Not necessarily related to this patch - yarn has ACLs enabled by default, MR 
has them disabled by default. That can lead to strange behavior considering 
some of the calls like killJob use Yarn ACLs, while getJobStatus will use MR 
ACLs. The Yarn ACLs could be disabled by default as well.

                
> no admin override to view jobs on mr app master and job history server
> ----------------------------------------------------------------------
>
>                 Key: MAPREDUCE-3903
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-3903
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: mrv2
>    Affects Versions: 0.23.2
>            Reporter: Thomas Graves
>            Assignee: Thomas Graves
>            Priority: Critical
>         Attachments: MAPREDUCE-3903.patch, MAPREDUCE-3903.patch
>
>
> in 1.0 there was a config mapreduce.cluster.administrators that allowed 
> administrators to view anyones job.  That no longer works on yarn.
> yarn has the new config yarn.admin.acl but it appears the mr app master and 
> job history server don't use that.  

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to