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

Sandy Ryza commented on YARN-1333:
----------------------------------

Thanks Tsuyoshi.  Just a couple more things:
{code}
+    resourceManager.getRMContext().getAMFinishingMonitor();
{code}
Is this line necessary?

{code}
+    ApplicationAttemptId appAttemptId = createSchedulingRequest(1024, 
"root.default", "user", 1);
{code}
This looks like more than 80 characters

{code}
+    if (SchedulerAppUtils.isBlacklisted(application, node, LOG)) {
+      return null;
+    }
{code}
Can this be moved to the equivalent of where it is in the capacity scheduler, 
i.e. FSLeafQueue?

> Support blacklisting in the Fair Scheduler
> ------------------------------------------
>
>                 Key: YARN-1333
>                 URL: https://issues.apache.org/jira/browse/YARN-1333
>             Project: Hadoop YARN
>          Issue Type: Improvement
>          Components: scheduler
>    Affects Versions: 2.2.0
>            Reporter: Sandy Ryza
>            Assignee: Tsuyoshi OZAWA
>         Attachments: YARN-1333.1.patch, YARN-1333.2.patch
>
>




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

Reply via email to