Github user leventov commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/490#discussion_r177043995
--- Diff: src/java/test/org/apache/zookeeper/test/AsyncHammerTest.java ---
@@ -61,6 +61,7 @@ protected void restart() throws Exception {
qb.startServers();
}
+ @SuppressWarnings("JUnit4TearDownNotRun")
--- End diff --
I would keep it as is for symmetry with setup() method, that couldn't be
declared `@Before` because it has a parameter.---
