[ https://issues.apache.org/jira/browse/FLINK-9735?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16532728#comment-16532728 ]
Sihua Zhou commented on FLINK-9735: ----------------------------------- Hi [~yuzhih...@gmail.com], I think this is a non issue, because the opt will be reused. If we have a look at the java doc of the {{OptionsFactory#createDBOptions()}} we can find that this is intended. {code:java} public interface OptionsFactory extends java.io.Serializable { /** * This method should set the additional options on top of the current options object. * The current options object may contain pre-defined options based on flags that have * been configured on the state backend. * * <p>It is important to set the options on the current object and return the result from * the setter methods, otherwise the pre-defined options may get lost. * * @param currentOptions The options object with the pre-defined options. * @return The options object on which the additional options are set. */ DBOptions createDBOptions(DBOptions currentOptions); /** * This method should set the additional options on top of the current options object. * The current options object may contain pre-defined options based on flags that have * been configured on the state backend. * * <p>It is important to set the options on the current object and return the result from * the setter methods, otherwise the pre-defined options may get lost. * * @param currentOptions The options object with the pre-defined options. * @return The options object on which the additional options are set. */ ColumnFamilyOptions createColumnOptions(ColumnFamilyOptions currentOptions); } {code} What do you think? > Potential resource leak in RocksDBStateBackend#getDbOptions > ----------------------------------------------------------- > > Key: FLINK-9735 > URL: https://issues.apache.org/jira/browse/FLINK-9735 > Project: Flink > Issue Type: Bug > Reporter: Ted Yu > Assignee: vinoyang > Priority: Minor > > Here is related code: > {code} > if (optionsFactory != null) { > opt = optionsFactory.createDBOptions(opt); > } > {code} > opt, an DBOptions instance, should be closed before being rewritten. > getColumnOptions has similar issue. -- This message was sent by Atlassian JIRA (v7.6.3#76005)