dcapwell commented on code in PR #3785:
URL: https://github.com/apache/cassandra/pull/3785#discussion_r1913642572
##########
test/harry/main/org/apache/cassandra/harry/model/PartitionState.java:
##########
@@ -223,7 +232,7 @@ private RowState updateRowState(RowState currentState,
IntFunction<Bijections.Bi
assert lts >= currentState.lts[i] :
String.format("Out-of-order LTS: %d. Max seen: %s", lts, currentState.lts[i]);
// sanity check; we're iterating in lts order
- if (currentState.lts[i] == lts)
+ if (lts != MagicConstants.NO_TIMESTAMP && currentState.lts[i]
== lts)
Review Comment:
the `ast` tests down track timestamps so this would always hit this branch
causing test failures... needed to disable this logic when doing a write and
just assume that the write is *newer*
--
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]
