Allow Skyframe graph lookups and value retrievals to throw InterruptedException.

The only place we now don't handle InterruptedException is in the action graph created after analysis, since I'm not sure that will be around for that much longer.

--
MOS_MIGRATED_REVID=130327770

1 files changed, 9 insertions, 11 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/actions/Action.java b/src/main/java/com/google/devtools/build/lib/actions/Action.java
index 01cf55472f..d7623de017 100644
--- a/src/main/java/com/google/devtools/build/lib/actions/Action.java
+++ b/src/main/java/com/google/devtools/build/lib/actions/Action.java
@@ -19,10 +19,8 @@ import com.google.devtools.build.lib.concurrent.ThreadSafety.ConditionallyThread
 import com.google.devtools.build.lib.profiler.Describable;
 import com.google.devtools.build.lib.vfs.Path;
 import com.google.devtools.build.lib.vfs.PathFragment;
-
 import java.io.IOException;
 import java.util.Collection;
-
 import javax.annotation.Nullable;
 
 /**
@@ -154,25 +152,25 @@ public interface Action extends ActionExecutionMetadata, Describable {
       throws ActionExecutionException, InterruptedException;
 
   /**
-   * Method used to resolve action inputs based on the information contained in
-   * the action cache. It will be called iff inputsKnown() is false for the
-   * given action instance and there is a related cache entry in the action
-   * cache.
+   * Method used to resolve action inputs based on the information contained in the action cache. It
+   * will be called iff inputsKnown() is false for the given action instance and there is a related
+   * cache entry in the action cache.
    *
-   * Method must be redefined for any action that may return
-   * inputsKnown() == false.
+   * <p>Method must be redefined for any action that may return inputsKnown() == false.
    *
    * @param artifactResolver the artifact factory that can be used to manufacture artifacts
    * @param resolver object which helps to resolve some of the artifacts
    * @param inputPaths List of relative (to the execution root) input paths
    * @return List of Artifacts corresponding to inputPaths, or null if some dependencies were
-   * missing and we need to try again later.
+   *     missing and we need to try again later.
    * @throws PackageRootResolutionException on failure to determine package roots of inputPaths
    */
   @Nullable
   Iterable<Artifact> resolveInputsFromCache(
-      ArtifactResolver artifactResolver, PackageRootResolver resolver,
-      Collection<PathFragment> inputPaths) throws PackageRootResolutionException;
+      ArtifactResolver artifactResolver,
+      PackageRootResolver resolver,
+      Collection<PathFragment> inputPaths)
+      throws PackageRootResolutionException, InterruptedException;
 
   /**
    * Informs the action that its inputs are {@code inputs}, and that its inputs are now known. Can