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

Hudson commented on YARN-4689:
------------------------------

FAILURE: Integrated in Hadoop-trunk-Commit #9317 (See 
[https://builds.apache.org/job/Hadoop-trunk-Commit/9317/])
YARN-4689. FairScheduler: Cleanup preemptContainer to be more readable. (kasha: 
rev 2ab4c476ed22d3ccf15b215710b67e32bbc7e5f0)
* 
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/FSParentQueue.java
* hadoop-yarn-project/CHANGES.txt
YARN-4689. Fix up CHANGES.txt (kasha: rev 
1c248ea4a8442bccc78a8ce4539ce1f4b6d6b0ee)
* hadoop-yarn-project/CHANGES.txt


> FairScheduler: Cleanup preemptContainer to be more readable
> -----------------------------------------------------------
>
>                 Key: YARN-4689
>                 URL: https://issues.apache.org/jira/browse/YARN-4689
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: fairscheduler
>            Reporter: Karthik Kambatla
>            Assignee: Kai Sasaki
>            Priority: Trivial
>              Labels: newbie
>         Attachments: YARN-4689.01.patch
>
>
> In FairScheduler#preemptContainer, we check if a queue is preemptable. The 
> code there could be cleaner if we don't use continue, but just the plain old 
> if-else. 



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

Reply via email to