Remove the unused method OutputMetadatStore.getTreeArtifactChildren().

This was accidentally omitted from 5107b62. RELNOTES: None. PiperOrigin-RevId: 691321242 Change-Id: Ibd0a3f8b6111f758c5f2e1f985b300b2e91a1ca2
bazelbuild · Oct 30, 2024 · adfd70e · adfd70e
1 parent b3ba592
commit adfd70e
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 20 deletions.
diff --git a/src/main/java/com/google/devtools/build/lib/actions/cache/OutputMetadataStore.java b/src/main/java/com/google/devtools/build/lib/actions/cache/OutputMetadataStore.java
@@ -13,11 +13,9 @@
 // limitations under the License.
 package com.google.devtools.build.lib.actions.cache;
 
-import com.google.common.collect.ImmutableSet;
 import com.google.devtools.build.lib.actions.ActionInput;
 import com.google.devtools.build.lib.actions.Artifact;
 import com.google.devtools.build.lib.actions.Artifact.SpecialArtifact;
-import com.google.devtools.build.lib.actions.Artifact.TreeFileArtifact;
 import com.google.devtools.build.lib.actions.FileArtifactValue;
 import com.google.devtools.build.lib.actions.FileStateType;
 import com.google.devtools.build.lib.skyframe.TreeArtifactValue;
@@ -49,12 +47,6 @@ public interface OutputMetadataStore extends MetadataInjector {
   /** Sets digest for virtual artifacts (e.g. middlemen). {@code digest} must not be null. */
   void setDigestForVirtualArtifact(Artifact artifact, byte[] digest);
 
-  /**
-   * Retrieves the children of a tree artifact, returning an empty set if there is no data
-   * available.
-   */
-  ImmutableSet<TreeFileArtifact> getTreeArtifactChildren(SpecialArtifact treeArtifact);
-
   /** Retrieves the metadata for this tree artifact. Data should already be available. */
   TreeArtifactValue getTreeArtifactValue(SpecialArtifact treeArtifact)
       throws IOException, InterruptedException;

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/ActionOutputMetadataStore.java b/src/main/java/com/google/devtools/build/lib/skyframe/ActionOutputMetadataStore.java
@@ -328,13 +328,6 @@ private TreeArtifactValue constructTreeArtifactValueFromFilesystem(SpecialArtifa
     return tree.build();
   }
 
-  @Override
-  public ImmutableSet<TreeFileArtifact> getTreeArtifactChildren(SpecialArtifact treeArtifact) {
-    checkArgument(treeArtifact.isTreeArtifact(), "%s is not a tree artifact", treeArtifact);
-    TreeArtifactValue tree = treeArtifactData.get(treeArtifact);
-    return tree != null ? tree.getChildren() : ImmutableSet.of();
-  }
-
   @Override
   public void injectFile(Artifact output, FileArtifactValue metadata) {
     checkArgument(isKnownOutput(output), "%s is not a declared output of this action", output);

diff --git a/src/test/java/com/google/devtools/build/lib/actions/util/ActionsTestUtil.java b/src/test/java/com/google/devtools/build/lib/actions/util/ActionsTestUtil.java
@@ -961,11 +961,6 @@ public void setDigestForVirtualArtifact(Artifact artifact, byte[] digest) {
       throw new UnsupportedOperationException();
     }
 
-    @Override
-    public ImmutableSet<TreeFileArtifact> getTreeArtifactChildren(SpecialArtifact treeArtifact) {
-      throw new UnsupportedOperationException();
-    }
-
     @Override
     public TreeArtifactValue getTreeArtifactValue(SpecialArtifact treeArtifact)
         throws IOException, InterruptedException {