saurabhagas commented on a change in pull request #2881:
URL: https://github.com/apache/iceberg/pull/2881#discussion_r679426943
##########
File path: api/src/main/java/org/apache/iceberg/io/ClosingIterator.java
##########
@@ -25,33 +25,30 @@
public class ClosingIterator<T> implements Iterator<T> {
Review comment:
If a premature close is required, the underlying `CloseableIterator`
should be used directly. I see this class as a convenience wrapper for the case
when all of the elements are intended to be consumed. I've added Javadoc
mentioning this.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
