Deletes default implemetation of MetadataProvider.getInput

Small misc cleanups.

PiperOrigin-RevId: 199797948

2 files changed, 3 insertions, 17 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/actions/MetadataProvider.java b/src/main/java/com/google/devtools/build/lib/actions/MetadataProvider.java
index 60db714d17..43938a732b 100644
--- a/src/main/java/com/google/devtools/build/lib/actions/MetadataProvider.java
+++ b/src/main/java/com/google/devtools/build/lib/actions/MetadataProvider.java
@@ -28,8 +28,8 @@ public interface MetadataProvider {
    * then t >= p. Aside from these properties, t can be any value and may vary arbitrarily across
    * calls.
    *
-   * <p>Returned {@link Metadata} instance correspond to the final target of a symlink, and
-   * therefore must not have a type of {@link FileStateType#SYMLINK} themselves.
+   * <p>Returned {@link FileArtifactValue} instance corresponds to the final target of a symlink and
+   * therefore must not have a type of {@link FileStateType#SYMLINK}.
    *
    * <p>The return value is owned by the cache and must not be modified.
    *
@@ -44,8 +44,5 @@ public interface MetadataProvider {
 
   /** Looks up an input from its exec path. */
   @Nullable
-  default ActionInput getInput(String execPath) {
-    // TODO(shahan): make sure all instances have an implementation and delete the default here.
-    return null;
-  }
+  ActionInput getInput(String execPath);
 }
diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/ActionFileSystem.java b/src/main/java/com/google/devtools/build/lib/skyframe/ActionFileSystem.java
index 03b1896ea8..92f51c4ee7 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/ActionFileSystem.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/ActionFileSystem.java
@@ -44,7 +44,6 @@ import java.io.InterruptedIOException;
 import java.io.OutputStream;
 import java.util.Collection;
 import java.util.HashMap;
-import java.util.concurrent.ConcurrentHashMap;
 import java.util.logging.Logger;
 import javax.annotation.Nullable;
 
@@ -66,7 +65,6 @@ final class ActionFileSystem extends FileSystem implements MetadataProvider, Inj
   private final FileSystem delegate;
 
   private final PathFragment execRootFragment;
-  private final Path execRootPath;
   private final ImmutableList<PathFragment> sourceRoots;
 
   private final ActionInputMap inputArtifactData;
@@ -74,9 +72,6 @@ final class ActionFileSystem extends FileSystem implements MetadataProvider, Inj
   /** exec path → artifact and metadata */
   private final HashMap<PathFragment, OptionalInputMetadata> optionalInputs;
 
-  /** digest → artifacts in {@link inputs} */
-  private final ConcurrentHashMap<ByteString, Artifact> optionalInputsByDigest;
-
   /** exec path → artifact and metadata */
   private final ImmutableMap<PathFragment, OutputMetadata> outputs;
 
@@ -103,7 +98,6 @@ final class ActionFileSystem extends FileSystem implements MetadataProvider, Inj
       this.delegate = delegate;
 
       this.execRootFragment = execRoot.asFragment();
-      this.execRootPath = getPath(execRootFragment);
       this.sourceRoots =
           sourceRoots
               .stream()
@@ -128,8 +122,6 @@ final class ActionFileSystem extends FileSystem implements MetadataProvider, Inj
             input.getExecPath(), unused -> new OptionalInputMetadata(input));
       }
 
-      this.optionalInputsByDigest = new ConcurrentHashMap<>();
-
       this.outputs =
           Streams.stream(outputArtifacts)
               .collect(
@@ -497,9 +489,6 @@ final class ActionFileSystem extends FileSystem implements MetadataProvider, Inj
             if (metadata == null) {
               throw new ActionExecutionFunction.MissingDepException();
             }
-            if (metadata.getType().exists() && metadata.getDigest() != null) {
-              optionalInputsByDigest.put(toByteString(metadata.getDigest()), artifact);
-            }
           }
         }
       }