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

Karthik Kambatla commented on YARN-1724:
----------------------------------------

Not a huge fan of adding a synchronization around a comparator, but looks like 
that would be the least invasive change in this case.

We should at least a very detailed comment explaining the reason for this 
synchronization.

> Race condition in Fair Scheduler when continuous scheduling is turned on 
> -------------------------------------------------------------------------
>
>                 Key: YARN-1724
>                 URL: https://issues.apache.org/jira/browse/YARN-1724
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: scheduler
>            Reporter: Sandy Ryza
>            Assignee: Sandy Ryza
>         Attachments: YARN-1724.patch
>
>
> If nodes resource allocations change during
>         Collections.sort(nodeIdList, nodeAvailableResourceComparator);
> we'll hit:
> java.lang.IllegalArgumentException: Comparison method violates its general 
> contract!



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to