[ https://issues.apache.org/jira/browse/FLINK-13245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16885280#comment-16885280 ]
zhijiang edited comment on FLINK-13245 at 7/15/19 2:27 PM: ----------------------------------------------------------- For the first issue, I think it is not very important currently, because we also has the {{PartitionRequestQueue#close}} to release all the view readers. For the second issue, it needs to fixed and I think we should still stick to the way of releasing partition in whole via {{ResultPartitionManager}}, that means we could not release one subpartition if the reference counter is not becoming zero in {{ReleaseOnConsumptionResultPartition}}. In detail: * In the process of {{BoundedBlockingSubpartitionReader#releaseAllResources}}, we should also call the {{parent.onConsumedSubpartition()}} as {{PipelinedSubpartitionView}} done in order to notify the {{ReleaseOnConsumptionResultPartition}} to decrease the reference counter. [~Zentol] [~azagrebin] [~StephanEwen] was (Author: zjwang): For the first issue, I think it is not very important currently, because we also has the {{PartitionRequestQueue#close }}to release all the view readers. For the second issue, it needs to fixed and I think we should still stick to the way of releasing partition in whole via {{ResultPartitionManager}}, that means we could not release one subpartition if the reference counter is not becoming zero in {{ReleaseOnConsumptionResultPartition}}. In detail: * In the process of {{BoundedBlockingSubpartitionReader#releaseAllResources}}, we should also call the {{parent.onConsumedSubpartition()}} as {{PipelinedSubpartitionView}} done in order to notify the {{ReleaseOnConsumptionResultPartition}} to decrease the reference counter. [~Zentol] [~azagrebin] [~StephanEwen] > Network stack is leaking files > ------------------------------ > > Key: FLINK-13245 > URL: https://issues.apache.org/jira/browse/FLINK-13245 > Project: Flink > Issue Type: Bug > Components: Runtime / Network > Affects Versions: 1.9.0 > Reporter: Chesnay Schepler > Assignee: zhijiang > Priority: Blocker > Fix For: 1.9.0 > > > There's file leak in the network stack / shuffle service. > When running the {{SlotCountExceedingParallelismTest}} on Windows a large > number of {{.channel}} files continue to reside in a > {{flink-netty-shuffle-XXX}} directory. > From what I've gathered so far these files are still being used by a > {{BoundedBlockingSubpartition}}. The cleanup logic in this class uses > ref-counting to ensure we don't release data while a reader is still present. > However, at the end of the job this count has not reached 0, and thus nothing > is being released. > The same issue is also present on the {{ResultPartition}} level; the > {{ReleaseOnConsumptionResultPartition}} also are being released while the > ref-count is greater than 0. > Overall it appears like there's some issue with the notifications for > partitions being consumed. > It is feasible that this issue has recently caused issues on Travis where the > build were failing due to a lack of disk space. -- This message was sent by Atlassian JIRA (v7.6.14#76016)