[ https://issues.apache.org/jira/browse/YARN-4225?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15018122#comment-15018122 ]
Jason Lowe commented on YARN-4225: ---------------------------------- Patch looks good overall, with just one minor issue regarding forward compatibility. If a newer client talks to an older ResourceManager then it will report a preemption status when none was provided. This could lead to some confusion when the preemption actually is disabled but the client defaults the missing field to false and reports preemption is enabled. Not sure that is must-fix for this scenario. If it is we'd have to expose something like the {{hasPreemptionDisabled}} protobuf method so the client could avoid reporting the value if the server didn't provide it. > Add preemption status to yarn queue -status for capacity scheduler > ------------------------------------------------------------------ > > Key: YARN-4225 > URL: https://issues.apache.org/jira/browse/YARN-4225 > Project: Hadoop YARN > Issue Type: Bug > Components: capacity scheduler, yarn > Affects Versions: 2.7.1 > Reporter: Eric Payne > Assignee: Eric Payne > Priority: Minor > Attachments: YARN-4225.001.patch, YARN-4225.002.patch > > -- This message was sent by Atlassian JIRA (v6.3.4#6332)