maedhroz commented on code in PR #2813:
URL: https://github.com/apache/cassandra/pull/2813#discussion_r1366226844
##########
test/distributed/org/apache/cassandra/distributed/test/IncRepairCoordinatorErrorTest.java:
##########
@@ -51,7 +51,7 @@ public void errorTest() throws Exception
cluster.get(3).runOnInstance(() -> {
ActiveRepairService.instance.failSession(result.toString(),
true);
});
- assertThat(cluster.get(1).logs().watchFor("Can't transition
endpoints .* to FAILED").getResult()).isNotEmpty();
+ assertThat(cluster.get(1).logs().watchFor("Removing completed
session .* with state FINALIZED").getResult()).isNotEmpty();
Review Comment:
nit: Any way we could just call into node 1 and get the number of sessions
via `ARS` rather than having to check the logs? If that's brutal, no biggie,
but I don't even thing we'd have to change the visibility of anything, and we
already have the `UUID`?
--
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]
