Remove special handling of children of tree artifacts from MinimalOutputStore.
RELNOTES: None.
@@ -25,8 +25,6 @@ import com.google.devtools.build.lib.actions.FileArtifactValue;
* <p>This store is intended for use with in-memory file systems, where aggressive caching would not
* be worthwhile.
*/
-// TODO(b/115361150): See if even less data can be retained. Specifically, we shouldn't need to
-// call injectRemoteFile for anything, including children of tree artifacts.
final class MinimalOutputStore extends OutputStore {
@Override
@@ -43,13 +41,5 @@ final class MinimalOutputStore extends OutputStore {
void addTreeArtifactContents(Artifact artifact, TreeFileArtifact contents) {}
@Override
- void injectRemoteFile(Artifact output, byte[] digest, long size, int locationIndex) {
- if (isChildOfTreeArtifact(output)) {
- super.injectRemoteFile(output, digest, size, locationIndex);
- }
- }
-
- private static boolean isChildOfTreeArtifact(Artifact artifact) {
- return artifact.hasParent() && artifact.getParent().isTreeArtifact();
- }
+ void injectRemoteFile(Artifact output, byte[] digest, long size, int locationIndex) {}
}
