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

Reply via email to