hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Eric Payne (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-2056) Disable preemption at Queue level
Date Tue, 09 Sep 2014 14:12:30 GMT

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

Eric Payne commented on YARN-2056:
----------------------------------

[~leftnoteasy], thank you very much for your review comments. I appreciate it.

Regarding the test for "appA running on queueB":
The following assertion tests that appA is preempted, but the preemption calculations are
done per queue.
{code}
+    verify(mDisp, times(10)).handle(argThat(new IsPreemptionRequestFor(appA)));
{code}
So, I added the following assertion to make the visual connection between appA and queueB:
{code}
+    assertTrue("appA should be running on queueB",
+        mCS.getAppsInQueue("queueB").contains(expectedAttemptOnQueueB));
{code}
So, the purpose is not really to test that the mockQueue/mockApp worked correctly, but to
make it obvious that the preemption policy for queueB is being exercised. If you think it's
not necessary, I will remove it, but I do like the link.

> Disable preemption at Queue level
> ---------------------------------
>
>                 Key: YARN-2056
>                 URL: https://issues.apache.org/jira/browse/YARN-2056
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: resourcemanager
>    Affects Versions: 2.4.0
>            Reporter: Mayank Bansal
>            Assignee: Eric Payne
>         Attachments: YARN-2056.201408202039.txt, YARN-2056.201408260128.txt, YARN-2056.201408310117.txt,
YARN-2056.201409022208.txt
>
>
> We need to be able to disable preemption at individual queue level



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

Mime
View raw message