Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/6147#discussion_r195060255
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/FileUploadHandler.java
---
@@ -89,7 +89,7 @@ protected void channelRead0(final ChannelHandlerContext
ctx, final HttpObject ms
final HttpContent httpContent = (HttpContent) msg;
currentHttpPostRequestDecoder.offer(httpContent);
- while (currentHttpPostRequestDecoder.hasNext()) {
--- End diff --
How exactly can this problem be reproduced? I don't fully understand why we
need to filter out `LastHttpContent`, because it should be consumable by the
`currentHttpPostRequestDecoder`. The only problem I can see from the code of
the `HttpPostMultipartRequestDecoder` is if we are in state
`MultiPartStatus.EPILOGUE` and then call `hasNext`.
---
