dcapwell commented on code in PR #4518:
URL: https://github.com/apache/cassandra/pull/4518#discussion_r2604069405
##########
test/harry/main/org/apache/cassandra/harry/model/PartitionState.java:
##########
@@ -410,6 +410,12 @@ public String toString(ValueGenerators valueGenerators)
", lts(" + StringUtils.toString(lts) + ")";
}
}
+
+ @Override
+ public String toString()
+ {
+ return toString(partitionState.valueGenerators);
Review Comment:
not related to this patch, but been in the other branch for ages...
when harry models detect a fault and show the rows, it was using the wrong
toString which means that the descriptors won't match making it hard to see
what happened...
--
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]
