dcapwell commented on code in PR #3656:
URL: https://github.com/apache/cassandra/pull/3656#discussion_r1830253658
##########
test/distributed/org/apache/cassandra/fuzz/sai/SingleNodeSAITestBase.java:
##########
@@ -116,6 +116,7 @@ public void basicSaiTest()
ColumnSpec.regularColumn("v3", ColumnSpec.int64Type)),
List.of(ColumnSpec.staticColumn("s1", ColumnSpec.asciiType(40, 100))),
withAccord ?
Optional.of(TransactionalMode.full) : Optional.empty())
+ .withWriteTimeFromAccord(false) // use the harry
timestamp
Review Comment:
spoke with Ariel to confirm we want this behavior for these tests. I don't
know *why* harry fails saying the values don't match what is expected... I
assume the writes are out of order but didn't confirm... According to Ariel
`USING TIMESTAMP` could be allowed and the visibility then degrades from strict
serializable and is up to the user...
TIL - This is allowed with `full`, only as long as you don't wrap in a
`BEGIN TRANSACTION`
--
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]
