Make a non-source artifact having a `SourceArtifactException` a hard crash instead of a bug report.
I see no recent history of this bug report, and it could have been explained by b/236308456.
@@ -932,12 +932,11 @@ public final class ActionExecutionFunction implements SkyFunction {
retrievedMetadata = nonMandatoryDiscovered.nextOrThrow(SourceArtifactException.class);
} catch (SourceArtifactException e) {
if (!input.isSourceArtifact()) {
- bugReporter.sendBugReport(
- new IllegalStateException(
+ throw new IllegalStateException(
String.format(
"Non-source artifact had SourceArtifactException %s %s",
input.toDebugString(), actionForError.prettyPrint()),
- e));
+ e);
}
skyframeActionExecutor.printError(e.getMessage(), actionForError);
