zhaomin1423 commented on code in PR #43502: URL: https://github.com/apache/spark/pull/43502#discussion_r1394277068
########## common/kvstore/src/main/java/org/apache/spark/util/kvstore/LevelDB.java: ########## @@ -322,26 +323,15 @@ public void close() throws IOException { } } - /** - * Closes the given iterator if the DB is still open. Trying to close a JNI LevelDB handle - * with a closed DB can cause JVM crashes, so this ensures that situation does not happen. - */ - void closeIterator(LevelDBIterator<?> it) throws IOException { - notifyIteratorClosed(it); - synchronized (this._db) { - DB _db = this._db.get(); - if (_db != null) { - it.close(); - } - } - } Review Comment: It was originally used in finalize method, but now it has been moved to ResourceCleaner.run. I think this logic is consistent -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org