Github user ifesdjeen commented on a diff in the pull request:
https://github.com/apache/cassandra/pull/257#discussion_r215442476
--- Diff: src/java/org/apache/cassandra/repair/RepairRunnable.java ---
@@ -651,7 +651,7 @@ private static void addRangeToNeighbors(List<CommonRange> neighborRangeList,
Ran
for (CommonRange cr : neighborRangeList)
{
// Use strict equality here, as worst thing that can happen is we generate
one more stream
- if (Iterables.elementsEqual(cr.endpoints, endpoints) && Iterables.elementsEqual(cr.transEndpoints,
transEndpoints))
+ if (Sets.difference(cr.endpoints, endpoints).isEmpty() && Sets.difference(cr.transEndpoints,
transEndpoints).isEmpty())
--- End diff --
Right, `return s.size() == o.size() && s.containsAll(o);` but with quick hashcode
path for non-equality.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org
|