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

    https://github.com/apache/flink/pull/4581#discussion_r148534322
  
    --- Diff: 
flink-runtime/src/test/java/org/apache/flink/runtime/io/network/partition/SpillableSubpartitionTest.java
 ---
    @@ -192,21 +198,39 @@ public void testConsumeSpilledPartition() throws 
Exception {
     
    --- End diff --
    
    nit: I'm evil I know, but could you replace mock `listener` with `new 
AwaitableBufferAvailablityListener()` if I please very nicely?


---

Reply via email to