divijvaidya commented on code in PR #14529: URL: https://github.com/apache/kafka/pull/14529#discussion_r1356912887
########## storage/src/main/java/org/apache/kafka/storage/internals/log/LazyIndex.java: ########## @@ -41,7 +41,7 @@ * Methods of this class are thread safe. Make sure to check `AbstractIndex` subclasses * documentation to establish their thread safety. */ -public class LazyIndex<T extends AbstractIndex> { +public class LazyIndex<T extends AbstractIndex> implements Closeable { Review Comment: `AutoCloseable` can be used with try-with-resource in Java whereas Closeable cannot. [1] https://docs.oracle.com/javase/tutorial/essential/exceptions/tryResourceClose.html -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org