mjsax commented on pull request #9028: URL: https://github.com/apache/kafka/pull/9028#issuecomment-660278609
Some follow up nits: The method `add10InputElements` does not throw any exception and we should remove the `throws` statement. Also in `AbstractResetIntegrationTest` in L283 we use `Assert.assertEquals(false, cleanResult)` that should be `Assert.assertFalse(cleanResult)` and L607 we use `Assert.assertEquals(true, cleanResult);` that should be `Assert.assertTrue(cleanResult);` ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org