You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Gary Yao (Jira)" <ji...@apache.org> on 2019/12/18 11:52:00 UTC

[jira] [Commented] (FLINK-14926) Make sure no resource leak of RocksObject

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

Gary Yao commented on FLINK-14926:
----------------------------------

What's the state here? There are already commits in release-1.10, e.g., 208de7c29f2ca71b526f98f603cf4a55103c5520.

> Make sure no resource leak of RocksObject
> -----------------------------------------
>
>                 Key: FLINK-14926
>                 URL: https://issues.apache.org/jira/browse/FLINK-14926
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Runtime / State Backends
>    Affects Versions: 1.6.4, 1.7.2, 1.8.2, 1.9.1
>            Reporter: Yu Li
>            Assignee: Yu Li
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> When investigating FLINK-14484 to allow setting {{WriteBufferManager}} via options, we find it necessary to supply a {{close}} method in {{OptionsFactory}} to make sure resources configured in options like {{WriteBufferManager}} could/would be released.
> What's more, we're having potential risk of resource leak in {{PredefinedOptions}} such as the created [BloomFilter instance|https://github.com/apache/flink/blob/e14bd50fed42e89674ba9d01a231d5c5e59f490c/flink-state-backends/flink-statebackend-rocksdb/src/main/java/org/apache/flink/contrib/streaming/state/PredefinedOptions.java#L155] is never closed (regression caused by FLINK-7220 changes), and we should also fix this and add a {{close}} method in {{PredefinedOptions}}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)