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

    https://github.com/apache/flink/pull/4552#discussion_r161547041
  
    --- Diff: 
flink-runtime/src/test/java/org/apache/flink/runtime/io/network/netty/PartitionRequestQueueTest.java
 ---
    @@ -71,4 +77,95 @@ public void testProducerFailedException() throws 
Exception {
                NettyMessage.ErrorResponse err = (NettyMessage.ErrorResponse) 
msg;
                assertTrue(err.cause instanceof CancelTaskException);
        }
    +
    +   /**
    +    * Tests {@link 
PartitionRequestQueue#enqueueAvailableReader(SequenceNumberingViewReader)},
    +    * verifying the reader would be enqueued in the pipeline if the next 
sending buffer is event, even
    --- End diff --
    
    nit: `is an event`


---

Reply via email to