Remove obsolete check.
@@ -24,7 +24,6 @@ import com.google.common.escape.Escaper;
import com.google.common.escape.Escapers;
import com.google.common.flogger.GoogleLogger;
import com.google.devtools.build.lib.actions.MutableActionGraph.ActionConflictException;
-import com.google.devtools.build.lib.bugreport.BugReport;
import com.google.devtools.build.lib.cmdline.Label;
import com.google.devtools.build.lib.packages.OutputFile;
import com.google.devtools.build.lib.skyframe.SkyframeAwareAction;
@@ -397,23 +396,6 @@ public final class Actions {
// possible from shared actions.
if (pathJ.getPathString().length() > pathI.getPathString().length()
&& pathJ.startsWith(pathI)) {
- // TODO(b/159733792): Test this check with compressed tree artifact input.
- // TODO(b/214351268): Remove this check.
- // We ignore the artifact prefix conflict between a TreeFileArtifact and its parent
- // TreeArtifact.
- // We can only have such a conflict here if the TreeArtifact is generated by an
- // ActionTemplate and the TreeFileArtifact is generated by an expanded action created at
- // execution time from the ActionTemplate.
- if (artifactJ.hasParent() && artifactJ.getParent().equals(artifactI)) {
- BugReport.sendBugReport(
- new IllegalStateException(
- String.format(
- "Impossible state: TreeFileArtifact %s (child of %s) shouldn't be available"
- + " before execution",
- artifactJ, artifactI)));
- continue;
- }
-
ActionAnalysisMetadata actionI =
Preconditions.checkNotNull(actionGraph.getGeneratingAction(artifactI), artifactI);
ActionAnalysisMetadata actionJ =
