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

Reply via email to