commit 5107b62a7dfe1e4f358940f6435190034bf6f472
author Googler <lberki@google.com> Mon Oct 28 09:45:58 2024 -0700
committer Copybara-Service <copybara-worker@google.com> Mon Oct 28 09:47:27 2024 -0700
tree 309e8776c6f5fad45c64961a5a0077ea20c9a23c
parent 6d0a834d560687e8d028516c6cef4f9d57f2e5a9

Remove OutputMetadataStore.getTreeArtifactChildren().

It was only a thin wrapper around .getTreeArtifactValue().

RELNOTES: None.
PiperOrigin-RevId: 690648081
Change-Id: I4b84ebc1a381ca8298dff2652ec60e00911294e0

src/main/java/com/google/devtools/build/lib/exec/StandaloneTestStrategy.java

diff --git a/src/main/java/com/google/devtools/build/lib/exec/StandaloneTestStrategy.java b/src/main/java/com/google/devtools/build/lib/exec/StandaloneTestStrategy.java
index 455f422..ff11db6 100644
--- a/src/main/java/com/google/devtools/build/lib/exec/StandaloneTestStrategy.java
+++ b/src/main/java/com/google/devtools/build/lib/exec/StandaloneTestStrategy.java
@@ -759,8 +759,8 @@
       ImmutableSet<? extends ActionInput> expandedCoverageDir =
           actionExecutionContext
               .getOutputMetadataStore()
-              .getTreeArtifactChildren(
-                  (SpecialArtifact) testAction.getCoverageDirectoryTreeArtifact());
+              .getTreeArtifactValue((SpecialArtifact) testAction.getCoverageDirectoryTreeArtifact())
+              .getChildren();
       ImmutableSet<ActionInput> coverageSpawnMetadata =
           ImmutableSet.<ActionInput>builder()
               .addAll(expandedCoverageDir)

src/main/java/com/google/devtools/build/lib/remote/AbstractActionInputPrefetcher.java

diff --git a/src/main/java/com/google/devtools/build/lib/remote/AbstractActionInputPrefetcher.java b/src/main/java/com/google/devtools/build/lib/remote/AbstractActionInputPrefetcher.java
index fbaed81..180308b 100644
--- a/src/main/java/com/google/devtools/build/lib/remote/AbstractActionInputPrefetcher.java
+++ b/src/main/java/com/google/devtools/build/lib/remote/AbstractActionInputPrefetcher.java
@@ -670,7 +670,8 @@
       }
 
       if (output.isTreeArtifact()) {
-        var children = outputMetadataStore.getTreeArtifactChildren((SpecialArtifact) output);
+        var children =
+            outputMetadataStore.getTreeArtifactValue((SpecialArtifact) output).getChildren();
         for (var file : children) {
           if (remoteOutputChecker.shouldDownloadOutput(file)) {
             outputsToDownload.add(file);

src/main/java/com/google/devtools/build/lib/remote/BazelOutputService.java

diff --git a/src/main/java/com/google/devtools/build/lib/remote/BazelOutputService.java b/src/main/java/com/google/devtools/build/lib/remote/BazelOutputService.java
index a626b0d..fbc7a8d 100644
--- a/src/main/java/com/google/devtools/build/lib/remote/BazelOutputService.java
+++ b/src/main/java/com/google/devtools/build/lib/remote/BazelOutputService.java
@@ -373,7 +373,8 @@
 
       if (output.isTreeArtifact()) {
         // TODO(chiwang): Use TreeArtifactLocator
-        var children = outputMetadataStore.getTreeArtifactChildren((SpecialArtifact) output);
+        var children =
+            outputMetadataStore.getTreeArtifactValue((SpecialArtifact) output).getChildren();
         for (var child : children) {
           addArtifact(outputMetadataStore, execRoot, outputPath, request, child);
         }

src/test/java/com/google/devtools/build/lib/skyframe/ActionOutputMetadataStoreTest.java

diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/ActionOutputMetadataStoreTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/ActionOutputMetadataStoreTest.java
index e846dec..1471e12 100644
--- a/src/test/java/com/google/devtools/build/lib/skyframe/ActionOutputMetadataStoreTest.java
+++ b/src/test/java/com/google/devtools/build/lib/skyframe/ActionOutputMetadataStoreTest.java
@@ -259,7 +259,7 @@
     assertThat(tree.getMetadata()).isEqualTo(treeMetadata);
     assertThat(tree.getChildValues())
         .containsExactly(child1, child1Metadata, child2, child2Metadata);
-    assertThat(store.getTreeArtifactChildren(treeArtifact)).isEqualTo(tree.getChildren());
+    assertThat(store.getTreeArtifactValue(treeArtifact)).isEqualTo(tree);
     assertThat(store.getAllArtifactData()).isEmpty();
     assertThat(chmodCalls).isEmpty();
   }
@@ -390,7 +390,7 @@
     assertThat(store.getAllTreeArtifactData().get(treeArtifact)).isEqualTo(tree);
     assertThat(chmodCalls).isEmpty();
 
-    assertThat(store.getTreeArtifactChildren(treeArtifact)).isEqualTo(tree.getChildren());
+    assertThat(store.getTreeArtifactValue(treeArtifact)).isEqualTo(tree);
 
     // Make sure that all children are transferred properly into the ActionExecutionValue. If any
     // child is missing, getExistingFileArtifactValue will throw.
@@ -699,7 +699,7 @@
     assertThat(tree.getMetadata()).isEqualTo(treeMetadata);
     assertThat(tree.getChildValues())
         .containsExactly(child1, child1Metadata, child2, child2Metadata);
-    assertThat(store.getTreeArtifactChildren(treeArtifact)).isEqualTo(tree.getChildren());
+    assertThat(store.getTreeArtifactValue(treeArtifact)).isEqualTo(tree);
     assertThat(store.getAllArtifactData()).isEmpty();
     assertThat(chmodCalls)
         .containsExactly(
@@ -714,15 +714,6 @@
   }
 
   @Test
-  public void getTreeArtifactChildren_noData_returnsEmptySet() {
-    SpecialArtifact treeArtifact =
-        ActionsTestUtil.createTreeArtifactWithGeneratingAction(
-            outputRoot, PathFragment.create("tree"));
-    ActionOutputMetadataStore store = createStore(/* outputs= */ ImmutableSet.of(treeArtifact));
-    assertThat(store.getTreeArtifactChildren(treeArtifact)).isEmpty();
-  }
-
-  @Test
   public void enteringExecutionModeClearsCachedOutputs() throws Exception {
     Artifact artifact =
         ActionsTestUtil.createArtifactWithRootRelativePath(