BEP: expose the number of cached test action in the summary

In the TestSummary, also indicate the total number of cached actions. Fixes bazelbuild#3435. Change-Id: I5fb3f54f54a852b7cbeb58b03b50b042e5d26455 PiperOrigin-RevId: 163871517
ManishSeal · Aug 2, 2017 · 0430492 · 0430492
1 parent b07f1ec
commit 0430492
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/src/main/java/com/google/devtools/build/lib/buildeventstream/proto/build_event_stream.proto b/src/main/java/com/google/devtools/build/lib/buildeventstream/proto/build_event_stream.proto
@@ -465,6 +465,9 @@ message TestSummary {
 
   // Path to logs of failed runs;
   repeated File failed = 4;
+
+  // Total number of cached test actions
+  int32 total_num_cached = 6;
 }
 
 // Event indicating the end of a build.

diff --git a/src/main/java/com/google/devtools/build/lib/runtime/TestSummary.java b/src/main/java/com/google/devtools/build/lib/runtime/TestSummary.java
@@ -470,6 +470,7 @@ public BuildEventStreamProtos.BuildEvent asStreamProto(BuildEventConverters conv
     BuildEventStreamProtos.TestSummary.Builder summaryBuilder =
         BuildEventStreamProtos.TestSummary.newBuilder()
             .setOverallStatus(BuildEventStreamerUtils.bepStatus(status))
+            .setTotalNumCached(getNumCached())
             .setTotalRunCount(totalRuns());
     for (Path path : getFailedLogs()) {
       summaryBuilder.addFailed(