Github user pnowojski commented on a diff in the pull request:
https://github.com/apache/flink/pull/6071#discussion_r191685431
--- Diff: pom.xml ---
@@ -300,15 +300,7 @@ under the License.
<dependency>
<groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-netty</artifactId>
- <!-- Don't upgrade for now. Netty versions >=
4.0.28.Final
- contain an improvement by Netty, which slices a
Netty buffer
- instead of doing a memory copy [1] in the
- LengthFieldBasedFrameDecoder. In some
situations, this
- interacts badly with our Netty pipeline leading
to OutOfMemory
- errors.
-
- [1] https://github.com/netty/netty/issues/3704
-->
-
<version>4.0.27.Final-${flink.shaded.version}</version>
+
<version>4.1.24.Final-${flink.shaded.version}</version>
--- End diff --
I hope that bumping to from `2.0` to `4.0` will not break other things, but
yes, that was my intention when `4.0` will be on mvn central.
---
