-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replicalayout followup #262
Conversation
…l, resulting in timeout instead of overload
ReplicaLayout.ForTokenWrite liveAndDown = ReplicaLayout.forTokenWriteLiveAndDown(keyspace, tk); | ||
Replicas.temporaryAssertFull(liveAndDown.all()); // TODO in CASSANDRA-14549 | ||
|
||
EndpointsForToken.Builder liveReplicasBuilder = EndpointsForToken.builder(tk); | ||
Replica selfReplica = liveAndDown.all().selfIfPresent(); | ||
if (selfReplica != null) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What exactly is this corner case where we have a hint written to ourselves? I thought we don't hint ourselves.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looked at mutateAtomically and sendToHintedEndpoints, I guess I can see that we do have the batch log attempt to write to the coordinator. I guess that sort of makes sense in the batch log doesn't really make sense sort of way. It's possible the coordinator didn't hand the write off to itself.
…A-16663) (apache#262) Ported from OSS commit d220d24.
…A-16663) (apache#262) Ported from OSS commit d220d24. (cherry picked from commit b762e3c)
…A-16663) (apache#262) Ported from OSS commit d220d24. (cherry picked from commit b762e3c)
…A-16663) (apache#262) Ported from OSS commit d220d24. (cherry picked from commit b762e3c) (cherry picked from commit 34374c1)
…A-16663) (apache#262) Ported from OSS commit d220d24. (cherry picked from commit b762e3c) (cherry picked from commit 34374c1) (cherry picked from commit b9046be)
No description provided.