I'm not saying closing the iterator is not needed. The same algorithm can be implemented using `Sequences.withBaggage()` , so `CloseableEnumerableIterator` itself is not needed. `Sequences.withBaggage()` wraps the underlying sequence and calls the given `closeable` after iterating all elements in the sequence.
[ Full content available at: https://github.com/apache/incubator-druid/pull/6094 ] This message was relayed via gitbox.apache.org for devnull@infra.apache.org