hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alejandro Abdelnur (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-1258) Allow configuring the Fair Scheduler root queue
Date Wed, 09 Oct 2013 03:32:44 GMT

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

Alejandro Abdelnur commented on YARN-1258:
------------------------------------------

LGTM, just 1 minor thing, instead doing:

{code}
      if (!(node instanceof Element)) {
        continue;
      }
{code}

I would do:

{code}
  if (node instanceof Element) {
   //ALL THE REST OF THE FORLOOP BLOCK
  }
{code}

+1 after this and a jenkins +1


> Allow configuring the Fair Scheduler root queue
> -----------------------------------------------
>
>                 Key: YARN-1258
>                 URL: https://issues.apache.org/jira/browse/YARN-1258
>             Project: Hadoop YARN
>          Issue Type: Improvement
>          Components: scheduler
>    Affects Versions: 2.1.1-beta
>            Reporter: Sandy Ryza
>            Assignee: Sandy Ryza
>         Attachments: YARN-1258.patch
>
>
> This would be useful for acls, maxRunningApps, scheduling modes, etc.
> The allocation file should be able to accept both:
> * An implicit root queue
> * A root queue at the top of the hierarchy with all queues under/inside of it



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

Mime
View raw message