belliottsmith commented on code in PR #4220: URL: https://github.com/apache/cassandra/pull/4220#discussion_r2240229230
########## test/simulator/main/org/apache/cassandra/simulator/Actions.java: ########## @@ -33,6 +33,24 @@ public class Actions { + public static class EmptyAction extends Action Review Comment: If this is a sentinel action for that specific use case, maybe move it there and give it a suitable name for the purpose? Doesn't seem to make much sense in isolation. -- 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: pr-unsubscr...@cassandra.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org For additional commands, e-mail: pr-h...@cassandra.apache.org