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

Yufei Gu commented on YARN-5831:
--------------------------------

There is an assumption from the original code: if the parent queue is 
non-preemptable, the children must be non-preemptable.  My patch follow this 
assumption.
{code}
    for (FSQueue q = getQueue();
        !q.getQueueName().equals("root");
        q = q.getParent()) {
      if (!q.isPreemptable()) {
        return false;
      }
    }
{code}

> Propagate allowPreemptionFrom flag all the way down to the app
> --------------------------------------------------------------
>
>                 Key: YARN-5831
>                 URL: https://issues.apache.org/jira/browse/YARN-5831
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: fairscheduler
>            Reporter: Karthik Kambatla
>            Assignee: Yufei Gu
>
> FairScheduler allows disallowing preemption from a queue. When checking if 
> preemption for an application is allowed, the new preemption code recurses 
> all the way to the root queue to check this flag. 
> Propagating this information all the way to the app will be more efficient. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to