[ 
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)

Reply via email to