Stanislav Chizhov created KAFKA-5985:
----------------------------------------

             Summary: 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
    Affects Versions: 0.11.0.0
            Reporter: Stanislav Chizhov


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 and 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)

Reply via email to