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.


---

Reply via email to