You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by "Damian Guy (JIRA)" <ji...@apache.org> on 2017/10/02 18:50:03 UTC
[jira] [Resolved] (KAFKA-5985) Mention the need to close store
iterators
[ https://issues.apache.org/jira/browse/KAFKA-5985?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Damian Guy resolved KAFKA-5985.
-------------------------------
Resolution: Fixed
Issue resolved by pull request 3994
[https://github.com/apache/kafka/pull/3994]
> Mention the need to close store iterators
> -----------------------------------------
>
> Key: KAFKA-5985
> URL: https://issues.apache.org/jira/browse/KAFKA-5985
> Project: Kafka
> Issue Type: Improvement
> Components: documentation, streams
> Affects Versions: 0.11.0.0
> Reporter: Stanislav Chizhov
> Assignee: Bill Bejeck
> Fix For: 1.0.0
>
>
> Store iterators should be closed in all/most of the cases, but currently it is not consistently reflected in the documentation and javadocs. For instance https://kafka.apache.org/0110/documentation/streams/developer-guide#streams_developer-guide_interactive-queries_custom-stores does not mention the need to close an iterator and provide an example that does not do that.
> Some of the fetch methods do mention the need to close an iterator returned (e.g. https://kafka.apache.org/0110/javadoc/org/apache/kafka/streams/state/ReadOnlyKeyValueStore.html#range(K,%20K)), but others do not: https://kafka.apache.org/0110/javadoc/org/apache/kafka/streams/state/ReadOnlyWindowStore.html#fetch(K,%20long,%20long)
> It makes sense to:
> - update javadoc for all store methods that do return iterators to reflect that the iterator returned needs to be closed
> - mention it in the documentation and to update related examples.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)