reschke commented on code in PR #2129: URL: https://github.com/apache/jackrabbit-oak/pull/2129#discussion_r1978718877
########## oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/directory/ActiveDeletedBlobCollectorFactory.java: ########## @@ -324,7 +323,11 @@ public void purgeBlobsDeleted(long before, @NotNull GarbageCollectableBlobStore long startBlobTrackerSyncTime = clock.getTime(); // Synchronize deleted blob ids with the blob id tracker try { - Closeables.close(idTempDeleteWriter, true); + try { Review Comment: I really don't think we need this. In this case, inlining actually makes things clearer. -- 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: oak-dev-unsubscr...@jackrabbit.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org