aweisberg commented on code in PR #4378:
URL: https://github.com/apache/cassandra/pull/4378#discussion_r2383289713
##########
src/java/org/apache/cassandra/locator/ReplicaPlans.java:
##########
@@ -632,6 +632,9 @@ E select(ConsistencyLevel consistencyLevel, L liveAndDown,
L live)
}
};
+ // TODO (review): Cheap quorums are not a goal for now, would really
require speculation for writes
+ // and would trigger reconciliation work later, potentially quite a lot so
I don't think it makes sense.
+ // So just remove selector?
Review Comment:
I'm going to leave this in for now to avoid disruption.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
