Github user aweisberg commented on a diff in the pull request:
https://github.com/apache/cassandra/pull/283#discussion_r225332967
--- Diff: src/java/org/apache/cassandra/locator/DynamicEndpointSnitch.java
---
@@ -188,6 +343,9 @@ public String getDatacenter(InetAddressAndPort endpoint)
public <C extends ReplicaCollection<? extends C>> C
sortedByProximity(final InetAddressAndPort address, C unsortedAddresses)
{
assert address.equals(FBUtilities.getBroadcastAddressAndPort());
// we only know about ourself
+ for (Replica unsortedAddress : unsortedAddresses)
--- End diff --
We shouldn't need to fetch each endpoint from the map a second time.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
