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 index e968b1bd1768c9..f52c0a61d26a6c 100644 --- 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 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 index 3056f1a4381338..2d9a1485da2ca2 100644 --- 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 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 index 827254ed5ebb29..607fc003f18474 100644 --- 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 getTreeArtifactChildren(SpecialArtifact treeArtifact) { - throw new UnsupportedOperationException(); - } - @Override public TreeArtifactValue getTreeArtifactValue(SpecialArtifact treeArtifact) throws IOException, InterruptedException {