Github user aweisberg commented on a diff in the pull request:
https://github.com/apache/cassandra/pull/224#discussion_r188443108
--- Diff:
src/java/org/apache/cassandra/service/AbstractWriteResponseHandler.java ---
@@ -225,7 +223,7 @@ protected boolean waitingFor(InetAddressAndPort from)
public void assureSufficientLiveNodes() throws UnavailableException
{
- consistencyLevel.assureSufficientLiveNodes(keyspace,
Iterables.filter(Iterables.concat(naturalEndpoints, pendingEndpoints),
isAlive));
+ consistencyLevel.assureSufficientLiveNodes(keyspace,
Replicas.filter(Replicas.concatNaturalAndPending(naturalReplicas,
pendingReplicas), isReplicaAlive));
--- End diff --
This ends up allocating two containers when all that is wanted is Iterables.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
