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

Karthik Kambatla commented on YARN-1241:
----------------------------------------

In the latest set of changes:
# If it is not too much of a bother, we should add a test to verify the 
correctness of counting non-runnable apps. 
# Would be nice to add a message to the exception thrown for better debugging:
{code}
+    @Override
+    public void remove() {
+      throw new UnsupportedOperationException();
+    }
{code}
# Might want to revisit this comment: find *and* gather? 
{code}
+  /**
+   * Traverses the queue hierarchy under the given queue to find gather
+   * all lists of non-runnable applications.
+   */
{code}
# Typo:
{code}
+   * We maintain positions in each of the lists.  Each next() call advances
+   * the position in one of the lists.  We maintain a heap that that orders 
lists
+   * by the start time of the app in the current position in that list.
{code}

> In Fair Scheduler maxRunningApps does not work for non-leaf queues
> ------------------------------------------------------------------
>
>                 Key: YARN-1241
>                 URL: https://issues.apache.org/jira/browse/YARN-1241
>             Project: Hadoop YARN
>          Issue Type: Bug
>    Affects Versions: 2.1.0-beta
>            Reporter: Sandy Ryza
>            Assignee: Sandy Ryza
>         Attachments: YARN-1241-1.patch, YARN-1241-10.patch, 
> YARN-1241-2.patch, YARN-1241-3.patch, YARN-1241-4.patch, YARN-1241-5.patch, 
> YARN-1241-6.patch, YARN-1241-7.patch, YARN-1241-8.patch, YARN-1241-9.patch, 
> YARN-1241.patch
>
>
> Setting the maxRunningApps property on a parent queue should make it that the 
> sum of apps in all subqueues can't exceed it



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

Reply via email to