[ https://issues.apache.org/jira/browse/YARN-3632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14546110#comment-14546110 ]
Craig Welch commented on YARN-3632: ----------------------------------- Uploaded patch to address checkstyle and whitespace concerns and to move queue acquisition from app inside the app syncronization. javac error unrelated, for an untouched class. {code} [WARNING] /home/jenkins/jenkins-slave/workspace/PreCommit-YARN-Build/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/TestRMRestart.java:[2171,32] [unchecked] Possible heap pollution from parameterized vararg type E [WARNING] where E is a type-variable: {code} Findbugs also appears unrelated, no changes appear to call into the area where the concern lies (it looks similar to the javac location, as though it missed a commit in the comparison??) > Ordering policy should be allowed to reorder an application when demand > changes > ------------------------------------------------------------------------------- > > Key: YARN-3632 > URL: https://issues.apache.org/jira/browse/YARN-3632 > Project: Hadoop YARN > Issue Type: Sub-task > Components: capacityscheduler > Reporter: Craig Welch > Assignee: Craig Welch > Attachments: YARN-3632.0.patch, YARN-3632.1.patch, YARN-3632.3.patch, > YARN-3632.4.patch > > > At present, ordering policies have the option to have an application > re-ordered (for allocation and preemption) when it is allocated to or a > container is recovered from the application. Some ordering policies may also > need to reorder when demand changes if that is part of the ordering > comparison, this needs to be made available (and used by the > fairorderingpolicy when sizebasedweight is true) -- This message was sent by Atlassian JIRA (v6.3.4#6332)