Remove duplicate TreeArtifactValue getSelfData method.
It is the same as getMetadata, so no need for two methods. For consistency, rename getSelfData in AggregatingArtifactValue to getMetadata as well.
RELNOTES: None.
PiperOrigin-RevId: 317159082
diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/ActionInputMapHelper.java b/src/main/java/com/google/devtools/build/lib/skyframe/ActionInputMapHelper.java
index 5f1d89e..a8d054f 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/ActionInputMapHelper.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/ActionInputMapHelper.java
@@ -72,9 +72,9 @@
inputMap,
/*depOwner=*/ key);
}
- // We have to cache the "digest" of the aggregating value itself,
- // because the action cache checker may want it.
- inputMap.put(key, aggregatingValue.getSelfData(), /*depOwner=*/ key);
+ // We have to cache the "digest" of the aggregating value itself, because the action cache
+ // checker may want it.
+ inputMap.put(key, aggregatingValue.getMetadata(), /*depOwner=*/ key);
// While not obvious at all this code exists to ensure that we don't expand the
// .runfiles/MANIFEST file into the inputs. The reason for that being that the MANIFEST
// file contains absolute paths that don't work with remote execution.
@@ -171,6 +171,6 @@
}
expandedArtifacts.put(treeArtifact, children.build());
// Again, we cache the "digest" of the value for cache checking.
- inputMap.put(treeArtifact, value.getSelfData(), depOwner);
+ inputMap.put(treeArtifact, value.getMetadata(), depOwner);
}
}
