Allow registering closeables if already closed (#7407)

atlasdb-impl-shared/src/main/java/com/palantir/atlasdb/transaction/impl/ThreadSafeCloser.java

@@ -17,7 +17,6 @@
 package com.palantir.atlasdb.transaction.impl;
 
 import com.google.common.io.Closer;
-import com.palantir.logsafe.exceptions.SafeIllegalStateException;
 import com.palantir.logsafe.logger.SafeLogger;
 import com.palantir.logsafe.logger.SafeLoggerFactory;
 import java.io.Closeable;
@@ -38,7 +37,7 @@ public synchronized <C extends Closeable> C register(C closeable) {
             } catch (IOException e) {
                 throw new RuntimeException(e);
             }
-            throw new SafeIllegalStateException("cannot register new closeable if already closed");
+            // throw new SafeIllegalStateException("cannot register new closeable if already closed");
         }
         return closer.register(closeable);
     }

atlasdb-impl-shared/src/test/java/com/palantir/atlasdb/transaction/impl/ThreadSafeCloserTest.java

@@ -22,6 +22,7 @@
 import com.palantir.logsafe.exceptions.SafeIllegalStateException;
 import java.io.Closeable;
 import java.io.IOException;
+import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 import org.mockito.Mockito;
 
@@ -37,6 +38,7 @@ void closes_onClose() throws IOException {
     }
 
     @Test
+    @Disabled
     void throws_ifAlreadyClosed() throws IOException {
         closer.close();
         Closeable closeable = Mockito.mock(Closeable.class);