Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/6071#discussion_r194666612
--- Diff:
flink-runtime/src/test/java/org/apache/flink/runtime/io/network/buffer/AbstractByteBufTest.java
---
@@ -64,12 +69,13 @@
import static org.junit.Assert.assertThat;
import static org.junit.Assert.assertTrue;
import static org.junit.Assert.fail;
+import static org.junit.Assume.assumeFalse;
import static org.junit.Assume.assumeTrue;
/**
* An abstract test class for channel buffers.
*
- * Copied from netty 4.0.50 with some changes to fit our netty version
4.0.27.
+ * Copy from netty 4.1.24.Final.
*/
public abstract class AbstractByteBufTest {
--- End diff --
Let's extend this class from `TestLogger`.
---
