[ 
https://issues.apache.org/jira/browse/FLINK-7699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16182815#comment-16182815
 ] 

ASF GitHub Bot commented on FLINK-7699:
---------------------------------------

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

    https://github.com/apache/flink/pull/4735#discussion_r141392529
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/consumer/RemoteInputChannel.java
 ---
    @@ -87,6 +88,12 @@
        /** The number of available buffers that have not been announced to the 
producer yet. */
        private final AtomicInteger unannouncedCredit = new AtomicInteger(0);
     
    +   /** The number of unsent buffers in the producer's sub partition. */
    +   private final AtomicInteger senderBacklog = new AtomicInteger(0);
    +
    +   /** The tag indicates whether this channel is waiting for additional 
floating buffers from the buffer pool. */
    +   private final AtomicBoolean isWaitingFloatingBuffers = new 
AtomicBoolean(false);
    --- End diff --
    
    better: `isWaitingForFloatingBuffers`


> Define the BufferListener interface to replace EventlListener in 
> BufferProvider
> -------------------------------------------------------------------------------
>
>                 Key: FLINK-7699
>                 URL: https://issues.apache.org/jira/browse/FLINK-7699
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Core
>            Reporter: zhijiang
>            Assignee: zhijiang
>             Fix For: 1.4.0
>
>
> Currently the {{EventListener}} is used in {{BufferProvider}} to be notified 
> of buffer available or destroyed pool. 
> To be semantic clearly, we define a new {{BufferListener}} interface which 
> can opt for a one-time only notification or to be notified repeatedly. And we 
> can also notify the pool destroyed via explicitly method 
> {{notifyBufferDestroyed}}.
> The {{RemoteInputChannel}} will implement the {{BufferListener}} interface to 
> wait for floating buffers from {{BufferProvider}}.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to