Tweak the default Skyframe thread count during testing. Since test cases are typically small, the current default is too high.

--
MOS_MIGRATED_REVID=91225981

1 files changed, 5 insertions, 1 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java b/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java
index c02e92d897..e3e954f434 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java
@@ -151,7 +151,11 @@ public abstract class SkyframeExecutor implements WalkableGraphFactory {
   // I just bumped this to 200 to get reasonable execution phase performance; that may cause
   // significant overhead for CPU-bound processes (i.e. analysis). [skyframe-analysis]
   @VisibleForTesting
-  public static final int DEFAULT_THREAD_COUNT = 200;
+  public static final int DEFAULT_THREAD_COUNT =
+      // Reduce thread count while running tests of Bazel. Test cases are typically small, and large
+      // thread pools vying for a relatively small number of CPU cores may induce non-optimal
+      // performance.
+      System.getenv("TEST_TMPDIR") == null ? 200 : 5;
 
   // Stores Packages between reruns of the PackageFunction (because of missing dependencies,
   // within the same evaluate() run) to avoid loading the same package twice (first time loading