PapaCharlie commented on code in PR #1989:
URL: https://github.com/apache/zookeeper/pull/1989#discussion_r1131325814
##########
zookeeper-server/src/test/java/org/apache/zookeeper/server/watch/WatcherCleanerTest.java:
##########
@@ -139,8 +141,10 @@ public void testMaxInProcessingDeadWatchers() {
assertTrue(listener.wait(5000));
}
- @Test
- public void testDeadWatcherMetrics() {
+ // There used to be a race condition surrounding this test which was
reproducible by running the test multiple
+ // times. This test is kept as repeated to flag if the race condition
reappears.
+ @RepeatedTest(5)
Review Comment:
Sure, up to you
--
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]
