commit 71b50f6b4c6e462863c8433a163c9eb23b0c5730
author janakr <janakr@google.com> Wed Jun 16 08:25:04 2021 -0700
committer Copybara-Service <copybara-worker@google.com> Wed Jun 16 08:25:58 2021 -0700
tree 606d9e5e189d32b95ecce5ee29720911207962e9
parent a53df66405ba687b5cc12736c81ffd2bd729dcea

Log when a URI future hasn't resolved by the time BuildToolLogs is converted into an event for the BEP.

PiperOrigin-RevId: 379724869

src/main/java/com/google/devtools/build/lib/buildeventstream/BuildToolLogs.java

diff --git a/src/main/java/com/google/devtools/build/lib/buildeventstream/BuildToolLogs.java b/src/main/java/com/google/devtools/build/lib/buildeventstream/BuildToolLogs.java
index 027e5a1..0fa1ad0 100644
--- a/src/main/java/com/google/devtools/build/lib/buildeventstream/BuildToolLogs.java
+++ b/src/main/java/com/google/devtools/build/lib/buildeventstream/BuildToolLogs.java
@@ -99,10 +99,9 @@
                 : null;
         if (uri != null) {
           toolLogs.addLog(
-              BuildEventStreamProtos.File.newBuilder()
-                  .setName(name)
-                  .setUri(Futures.getDone(directFuture))
-                  .build());
+              BuildEventStreamProtos.File.newBuilder().setName(name).setUri(uri).build());
+        } else {
+          logger.atInfo().log("Dropped unfinished upload: %s (%s)", name, directFuture);
         }
       } catch (ExecutionException e) {
         logger.atWarning().withCause(e).log("Skipping build tool log upload %s", name);