Github user pnowojski commented on a diff in the pull request:
https://github.com/apache/flink/pull/6071#discussion_r191700895
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/io/network/buffer/ReadOnlySlicedNetworkBuffer.java
---
@@ -75,12 +75,12 @@ private ReadOnlySlicedNetworkBuffer(ByteBuf buffer, int
index, int length, int m
@Override
public ByteBuf unwrap() {
- return super.unwrap().unwrap();
+ return super.unwrap();
--- End diff --
Necessary for the upgrade, otherwise our tests (including `ITCases`) are
failing. In upgraded version there is a different structure of nested classes
in case of using slices of readonly buffers. Possibly this is a bug fix for a
bug, that didn't show up in older netty.
---
