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

Bikas Saha commented on YARN-881:
---------------------------------

That internal code should probably create its own comparator. This compareTo 
method for the class is user facing and its inconsistent for users to see the 
compareTo() method returning results that are opposite to the declared ordering 
of priorities in yarn. [~vinodkv] - what do you think?
                
> Priority#compareTo method seems to be wrong.
> --------------------------------------------
>
>                 Key: YARN-881
>                 URL: https://issues.apache.org/jira/browse/YARN-881
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Jian He
>            Assignee: Jian He
>
> if lower int value means higher priority, shouldn't we "return 
> other.getPriority() - this.getPriority() " 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to