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

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

Github user zhijiangW commented on the issue:

    https://github.com/apache/flink/pull/6238
  
    Thanks for fixing this bug. :)
    I think there is another potential bug in polling segments from queue 
during task canceling process, and I pointed out it in the above codes.


> Network buffer leaks in requesting a batch of segments during canceling
> -----------------------------------------------------------------------
>
>                 Key: FLINK-9636
>                 URL: https://issues.apache.org/jira/browse/FLINK-9636
>             Project: Flink
>          Issue Type: Bug
>          Components: Network
>    Affects Versions: 1.5.0, 1.6.0
>            Reporter: zhijiang
>            Assignee: Nico Kruber
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.5.1
>
>
> In {{NetworkBufferPool#requestMemorySegments}}, {{numTotalRequiredBuffers}} 
> is increased by {{numRequiredBuffers}} first.
> If {{InterruptedException}} is thrown during polling segments from the 
> available queue, the requested segments will be recycled back to 
> {{NetworkBufferPool}}, {{numTotalRequiredBuffers}} is decreased by the number 
> of polled segments which is now inconsistent with {{numRequiredBuffers}}. So 
> {{numTotalRequiredBuffers}} in {{NetworkBufferPool}} leaks in this case, and 
> we can also decrease {{numRequiredBuffers}} to fix this bug.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to