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

Hudson commented on MAPREDUCE-7199:
-----------------------------------

SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #18160 (See 
[https://builds.apache.org/job/Hadoop-trunk-Commit/18160/])
MAPREDUCE-7199. HsJobsBlock reuse JobACLsManager for checkAccess. 
(surendralilhore: rev a1b0697d379d33223ec1a46dfef31d6d226169bb)
* (edit) 
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/JobACLsManager.java
* (edit) 
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-hs/src/main/java/org/apache/hadoop/mapreduce/v2/hs/webapp/HsJobsBlock.java


> HsJobsBlock reuse JobACLsManager for checkAccess
> ------------------------------------------------
>
>                 Key: MAPREDUCE-7199
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-7199
>             Project: Hadoop Map/Reduce
>          Issue Type: Improvement
>            Reporter: Bibin Chundatt
>            Assignee: Bilwa S T
>            Priority: Minor
>             Fix For: 3.4.0
>
>         Attachments: MAPREDUCE-7199-001.patch, MAPREDUCE-7199.002.patch, 
> MAPREDUCE-7199.003.patch
>
>
> Reuse JobAclManager.checkAccess
> {code} 
>  private boolean checkAccess(String userName) {
>     if(!areAclsEnabled) {
>       return true;
>     }
>     // User could see its own job.
>     if (ugi.getShortUserName().equals(userName)) {
>       return true;
>     }
>     // Admin could also see all jobs
>     if (adminAclList != null && adminAclList.isUserAllowed(ugi)) {
>       return true;
>     }
>     return false;
>   }
> {code} 
> {code}
> jobACLsManager
>           .checkAccess(ugi, JobACL.VIEW_JOB, ..
>               new AccessControlList()))
> {code}



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

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

Reply via email to