From 6c3ac8acf02a6fa5758dc8e67b7f6c9a428e4e67 Mon Sep 17 00:00:00 2001
From: Janak Ramakrishnan <janakr@google.com>
Date: Thu, 17 Nov 2016 18:30:46 +0000
Subject: Don't get output service or action input prefetcher if not running a
 "build" command, to save time on the critical path of a build.

--
MOS_MIGRATED_REVID=139477157
---
 .../build/lib/runtime/CommandEnvironment.java      | 29 ++++++++++++----------
 1 file changed, 16 insertions(+), 13 deletions(-)

(limited to 'src/main/java')

diff --git a/src/main/java/com/google/devtools/build/lib/runtime/CommandEnvironment.java b/src/main/java/com/google/devtools/build/lib/runtime/CommandEnvironment.java
index d6d0c485a6..264fa22195 100644
--- a/src/main/java/com/google/devtools/build/lib/runtime/CommandEnvironment.java
+++ b/src/main/java/com/google/devtools/build/lib/runtime/CommandEnvironment.java
@@ -534,21 +534,24 @@ public final class CommandEnvironment {
     outputService = null;
     BlazeModule outputModule = null;
     ImmutableList.Builder<ActionInputPrefetcher> prefetchersBuilder = ImmutableList.builder();
-    for (BlazeModule module : runtime.getBlazeModules()) {
-      OutputService moduleService = module.getOutputService();
-      if (moduleService != null) {
-        if (outputService != null) {
-          throw new IllegalStateException(String.format(
-              "More than one module (%s and %s) returns an output service",
-              module.getClass(), outputModule.getClass()));
+    if (command.builds()) {
+      for (BlazeModule module : runtime.getBlazeModules()) {
+        OutputService moduleService = module.getOutputService();
+        if (moduleService != null) {
+          if (outputService != null) {
+            throw new IllegalStateException(
+                String.format(
+                    "More than one module (%s and %s) returns an output service",
+                    module.getClass(), outputModule.getClass()));
+          }
+          outputService = moduleService;
+          outputModule = module;
         }
-        outputService = moduleService;
-        outputModule = module;
-      }
 
-      ActionInputPrefetcher actionInputPrefetcher = module.getPrefetcher();
-      if (actionInputPrefetcher != null) {
-        prefetchersBuilder.add(actionInputPrefetcher);
+        ActionInputPrefetcher actionInputPrefetcher = module.getPrefetcher();
+        if (actionInputPrefetcher != null) {
+          prefetchersBuilder.add(actionInputPrefetcher);
+        }
       }
     }
     final ImmutableList<ActionInputPrefetcher> actionInputPrefetchers = prefetchersBuilder.build();
-- 
cgit v1.2.3