commit 16274e4f00c91a2a12fe51e397254493be3f355d
author janakr <janakr@google.com> Tue Nov 13 13:04:14 2018 -0800
committer Copybara-Service <copybara-piper@google.com> Tue Nov 13 13:07:02 2018 -0800
tree 87dd9540d92c428f383d03ba6581d026b4158eb5
parent baa1786fe6ac01511bb15689a81b2978f39686da

Allow checking of critical path inconsistencies to be overridden, and remove now unused discardActions parameter to CriticalPathComputer.

PiperOrigin-RevId: 221319368

src/main/java/com/google/devtools/build/lib/runtime/BuildSummaryStatsModule.java

diff --git a/src/main/java/com/google/devtools/build/lib/runtime/BuildSummaryStatsModule.java b/src/main/java/com/google/devtools/build/lib/runtime/BuildSummaryStatsModule.java
index cd782bd..9f237d4 100644
--- a/src/main/java/com/google/devtools/build/lib/runtime/BuildSummaryStatsModule.java
+++ b/src/main/java/com/google/devtools/build/lib/runtime/BuildSummaryStatsModule.java
@@ -50,7 +50,6 @@
   private EventBus eventBus;
   private Reporter reporter;
   private boolean enabled;
-  private boolean discardActions;
 
   private SpawnStats spawnStats;
 
@@ -74,14 +73,12 @@
   @Override
   public void executorInit(CommandEnvironment env, BuildRequest request, ExecutorBuilder builder) {
     enabled = env.getOptions().getOptions(ExecutionOptions.class).enableCriticalPathProfiling;
-    discardActions = !env.getSkyframeExecutor().tracksStateForIncrementality();
   }
 
   @Subscribe
   public void executionPhaseStarting(ExecutionStartingEvent event) {
     if (enabled) {
-      criticalPathComputer =
-          new CriticalPathComputer(actionKeyContext, BlazeClock.instance(), discardActions);
+      criticalPathComputer = new CriticalPathComputer(actionKeyContext, BlazeClock.instance());
       eventBus.register(criticalPathComputer);
     }
   }