[jira] [Commented] (YARN-5831) Propagate allowPreemptionFrom flag all the way down to the app

["Yufei Gu (JIRA)" <ji...@apache.org>]

    [ 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: yarn-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: yarn-issues-help@hadoop.apache.org