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

    https://github.com/apache/flink/pull/5105#discussion_r156426121
  
    --- Diff: 
flink-runtime/src/test/java/org/apache/flink/runtime/io/network/util/TestPooledBufferProvider.java
 ---
    @@ -28,66 +29,62 @@
     
     import java.io.IOException;
     import java.util.Queue;
    -import java.util.concurrent.ArrayBlockingQueue;
    +import java.util.concurrent.BlockingQueue;
     import java.util.concurrent.ConcurrentLinkedQueue;
    +import java.util.concurrent.LinkedBlockingDeque;
     
     import static org.apache.flink.util.Preconditions.checkArgument;
     
     public class TestPooledBufferProvider implements BufferProvider {
     
        private final Object bufferCreationLock = new Object();
    --- End diff --
    
    this lock is unused now


---

Reply via email to