dcapwell commented on code in PR #3694:
URL: https://github.com/apache/cassandra/pull/3694#discussion_r1854292284
##########
src/java/org/apache/cassandra/service/consensus/TransactionalMode.java:
##########
@@ -112,7 +112,10 @@ public enum TransactionalMode
* Execute writes through Accord skipping StorageProxy's normal write
path. Ignores the provided consistency level
* which makes Accord commit writes at ANY similar to Paxos with commit
consistency level ANY.
*/
- full(true, true, true, true, true);
+ full(true, true, true, true, true),
+
+ // For tests, Accord will read and be forced to do interop reads
+ test_interop_read(true, false, false, true, true);
Review Comment:
what is the reason for having test only states? It's more testing that
should never end up in prod, so what is the value we get? Where do we test
this really (it looks like we just take existing tests and run with these
modes; so not really testing these modes)
--
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]
