Github user pnowojski commented on a diff in the pull request:

    https://github.com/apache/flink/pull/4517#discussion_r136831143
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/io/network/netty/NettyMessage.java
 ---
    @@ -378,11 +373,14 @@ ByteBuf write(ByteBufAllocator allocator) throws 
IOException {
                        }
                }
     
    -           @Override
    -           public void readFrom(ByteBuf buffer) {
    -                   partitionId = new 
ResultPartitionID(IntermediateResultPartitionID.fromByteBuf(buffer), 
ExecutionAttemptID.fromByteBuf(buffer));
    -                   queueIndex = buffer.readInt();
    -                   receiverId = InputChannelID.fromByteBuf(buffer);
    +           static PartitionRequest readFrom(ByteBuf buffer) {
    +                   PartitionRequest result = new PartitionRequest();
    --- End diff --
    
    ditto


---

Reply via email to