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

    https://github.com/apache/flink/pull/4552#discussion_r161573976
  
    --- Diff: 
flink-runtime/src/test/java/org/apache/flink/runtime/io/network/partition/SpillableSubpartitionTest.java
 ---
    @@ -238,6 +238,7 @@ public void testConsumeSpilledPartition() throws 
Exception {
     
                verify(listener, times(1)).notifyBuffersAvailable(eq(4L));
     
    +           assertFalse(reader.nextBufferIsEvent());
    --- End diff --
    
    also test `read.buffer().isBuffer()`?


---

Reply via email to