belliottsmith commented on code in PR #163: URL: https://github.com/apache/cassandra-accord/pull/163#discussion_r1939124111
########## accord-core/src/main/java/accord/coordinate/CoordinateGloballyDurable.java: ########## @@ -31,12 +31,12 @@ import accord.utils.async.AsyncResult; import accord.utils.async.AsyncResults.SettableResult; -// TODO (expected): this does not need to query every shard; can disseminate globally any sub-range of the ring -// (indeed, we could slice both the query and dissemination only so that they always overlap) +// TODO (expected): this does not need to query every shard OR more than one replica per shard; +// can disseminate globally any sub-range of the ring. +// Infact, we could simply autonomously disseminate our latest information to some subset of replicas Review Comment: Good question. I think perhaps, otherwise I probably wouldn't have done it? But I didn't consciously decide to, so who knows. It looks like "official" versions have a space, but English doesn't actually have any official official version, so 🤷 -- 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]
