Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/6333#discussion_r202553358
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/state/TieBreakingPriorityComparator.java
---
@@ -85,10 +85,10 @@ public int compare(T o1, T o2) {
return ((Comparable<T>) o1).compareTo(o2);
}
- // we catch this case before moving to more expensive tie
breaks.
- if (o1.equals(o2)) {
- return 0;
- }
+// // we catch this case before moving to more expensive tie
breaks.
--- End diff --
I think this is some commented out code which should be removed.
---
