commit 8bd64541ab283a4e3f4c3b97e7df49fb06da4e60
author emilyguo <emilyguo@google.com> Fri Apr 01 08:11:51 2022 -0700
committer Copybara-Service <copybara-worker@google.com> Fri Apr 01 08:13:13 2022 -0700
tree 6afad0b105a2f1b714bb2703828bde666001ea86
parent e133373b489a9ece913893c56098cd244c3a092a

Change to a lower-noise reporting method that fails tests.

It can avoid to keep sending Third eye reports, but still get error message to help further investigation.

PiperOrigin-RevId: 438826738

src/main/java/com/google/devtools/build/lib/rules/genquery/GenQuery.java

diff --git a/src/main/java/com/google/devtools/build/lib/rules/genquery/GenQuery.java b/src/main/java/com/google/devtools/build/lib/rules/genquery/GenQuery.java
index 1f681f0..b47ae91 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/genquery/GenQuery.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/genquery/GenQuery.java
@@ -254,9 +254,7 @@
     for (SkyKey skyKey : transitiveTargetKeys) {
       SkyValue value = transitiveTargetValues.next();
       if (value == null) {
-        BugReport.sendBugReport(
-            new IllegalStateException(
-                "SkyValue " + skyKey + " was missing, this should never happen"));
+        BugReport.logUnexpected("Value for: '%s' was missing, this should never happen", skyKey);
         return null;
       }
       TransitiveTargetValue transNode = (TransitiveTargetValue) value;