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

Alejandro Abdelnur commented on YARN-1288:
------------------------------------------

* Queue.java: 
** false change line 46 
** unused import line 27
* FSLeafQueue.java: 
** unused imports lines 27,34
* FSQueue.java: 
** unused imports lines 23,24,29
* QueueManager.java: 
** false changes line 75,296,414,417
** NOONE_ACL constant should be NO_ONE_ACL
** default getQueueAcl() behavior has not changed, correct? The following 
comment is removed by the patch, I think we should have it in the getQueueAcl() 
method:

{code}
       // Root queue should have empty ACLs.  As a queue's ACL is the union of
      // its ACL and all its parents' ACLs, setting the roots' to empty will
      // neither allow nor prohibit more access to its children.
{code}


> Make Fair Scheduler ACLs more user friendly
> -------------------------------------------
>
>                 Key: YARN-1288
>                 URL: https://issues.apache.org/jira/browse/YARN-1288
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: scheduler
>    Affects Versions: 2.2.0
>            Reporter: Sandy Ryza
>            Assignee: Sandy Ryza
>         Attachments: YARN-1288.patch
>
>
> The Fair Scheduler currently defaults the root queue's acl to empty and all 
> other queues' acl to "*".  Now that YARN-1258 enables configuring the root 
> queue, we should reverse this.  This will also bring the Fair Scheduler in 
> line with the Capacity Scheduler.
> We should also not trim the acl strings, which makes it impossible to only 
> specify groups in an acl.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to