aweisberg commented on code in PR #4447:
URL: https://github.com/apache/cassandra/pull/4447#discussion_r2683924654
##########
src/java/org/apache/cassandra/replication/TrackedWriteRequest.java:
##########
@@ -224,6 +224,9 @@ public static void applyLocallyAndSendToReplicas(Mutation
mutation, ReplicaPlan.
static void applyMutationLocally(Mutation mutation,
RequestCallback<NoPayload> handler)
{
Preconditions.checkArgument(handler instanceof
TrackedWriteResponseHandler || handler instanceof
ForwardedWrite.LeaderCallback);
+ // TODO: Why execute immediately before even sending the messages to
the other nodes?
+ // Also this is already going to be on the mutation stage and the
mutation stage can get backed up so don't we want
+ // to start on a stage that is less likey to get backed up?
Review Comment:
Basically same issue as noted in Verb
--
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]
