[Skymeld] Make the shutting down of various executors interruptible.

... since there's no reason for them to be uninterruptible in the first place.

PiperOrigin-RevId: 530580587
Change-Id: I9c7848f9d09599583cb7ca33f6397a0e87f9123b

Author: joeleba

src/main/java/com/google/devtools/build/lib/skyframe/IncrementalArtifactConflictFinder.java

@@ -125,9 +125,10 @@ private static void constructActionGraphAndArtifactList(
     }
   }
 
-  void shutdown() throws InterruptedException {
-    if (!executorService.isShutdown() && ExecutorUtil.uninterruptibleShutdown(executorService)) {
-      throw new InterruptedException();
+  void shutdown() {
+    if (!executorService.isShutdown() && ExecutorUtil.interruptibleShutdown(executorService)) {
+      // Preserve the interrupt status.
+      Thread.currentThread().interrupt();
     }
   }
 

src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java

@@ -2492,7 +2492,7 @@ public void resetBuildDriverFunction() {
   }
 
   /** Resets the incremental artifact conflict finder to ensure incremental correctness. */
-  public void resetIncrementalArtifactConflictFindingStates() throws InterruptedException {
+  public void resetIncrementalArtifactConflictFindingStates() {
     incrementalArtifactConflictFinder.shutdown();
     incrementalTransitiveActionLookupKeysCollector.shutdown();
     incrementalArtifactConflictFinder =
@@ -3856,9 +3856,10 @@ private boolean tryClaimVisitation(
           && collected.putIfAbsent(key, ClaimedLookupValueSentinel.INSTANCE) == null;
     }
 
-    private void shutdown() throws InterruptedException {
-      if (!executorService.isShutdown() && ExecutorUtil.uninterruptibleShutdown(executorService)) {
-        throw new InterruptedException();
+    private void shutdown() {
+      if (!executorService.isShutdown() && ExecutorUtil.interruptibleShutdown(executorService)) {
+        // Preserve the interrupt status.
+        Thread.currentThread().interrupt();
       }
     }