dcapwell commented on code in PR #3656: URL: https://github.com/apache/cassandra/pull/3656#discussion_r1828432070
########## test/distributed/org/apache/cassandra/fuzz/topology/AccordTopologyMixupTest.java: ########## @@ -203,15 +201,15 @@ public String keyspaceName() } } - private static class AccordState extends State<Spec> implements SharedState.Listener + private static class AccordState extends State<Spec> { - private final List<Runnable> onError = new CopyOnWriteArrayList<>(); + private final ListenerHolder listener; Review Comment: to enable `HarryOnAccord` without a ton of copy paste, refactor so the logic can be shared -- 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