lwhite1 commented on PR #14534: URL: https://github.com/apache/arrow/pull/14534#issuecomment-1295166591
> LGTM. > > Is there somewhere in the Javadocs or prose docs that this behavior should be described? (That is: if the VSR is reused, closing the iterator closes the root as expected; else the caller is expected to manage all roots) Good catch. It is mentioned in the method javadoc for next(). I just pushed a commit that makes it explicit in the close() method. -- 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]
