Github user aweisberg commented on a diff in the pull request:
https://github.com/apache/cassandra/pull/257#discussion_r215434653
--- 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 --
You can use .equals() here. Set equality does the right thing.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org
