diff options
| author |  ulfjack <ulfjack@google.com> | 2018-07-26 09:40:58 -0700 |
|---|---|---|
| committer |  Copybara-Service <copybara-piper@google.com> | 2018-07-26 09:42:29 -0700 |
| commit | e24a591d4288b33dfbf790e89e0def610fe6a7d9 (patch) | |
| tree | 7499dcd1e9ef99257966d7fc9c6e768eb1622b54 /src/test/java | |
| parent | 09efb3fb5cbefe48d7c6685b01cfd72f848563c8 (diff) | |
Target pattern eval: use the same number of threads as loading
Instead of using the default thread pool size of 200, use the number set for
the loading phase. This is in preparation for interleaving the loading and
target pattern eval phases.
PiperOrigin-RevId: 206172915
Diffstat (limited to 'src/test/java')
3 files changed, 4 insertions, 0 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/util/AnalysisTestCase.java b/src/test/java/com/google/devtools/build/lib/analysis/util/AnalysisTestCase.java index 22d8fff15c..11fe5375b4 100644 --- a/src/test/java/com/google/devtools/build/lib/analysis/util/AnalysisTestCase.java +++ b/src/test/java/com/google/devtools/build/lib/analysis/util/AnalysisTestCase.java @@ -353,6 +353,7 @@ public abstract class AnalysisTestCase extends FoundationTestCase { ImmutableList.copyOf(labels), PathFragment.EMPTY_FRAGMENT, loadingOptions, + LOADING_PHASE_THREADS, keepGoing, /*determineTests=*/ false); diff --git a/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java b/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java index 00acc720f7..1f9738148e 100644 --- a/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java +++ b/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java @@ -1746,6 +1746,7 @@ public abstract class BuildViewTestCase extends FoundationTestCase { targets, PathFragment.EMPTY_FRAGMENT, loadingOptions, + loadingPhaseThreads, keepGoing, /*determineTests=*/ false); if (!doAnalysis) { diff --git a/src/test/java/com/google/devtools/build/lib/pkgcache/LoadingPhaseRunnerTest.java b/src/test/java/com/google/devtools/build/lib/pkgcache/LoadingPhaseRunnerTest.java index cd9246eee9..50e6ecf0e6 100644 --- a/src/test/java/com/google/devtools/build/lib/pkgcache/LoadingPhaseRunnerTest.java +++ b/src/test/java/com/google/devtools/build/lib/pkgcache/LoadingPhaseRunnerTest.java @@ -815,6 +815,8 @@ public class LoadingPhaseRunnerTest { ImmutableList.copyOf(patterns), PathFragment.EMPTY_FRAGMENT, options, + // We load very few packages, and everything is in memory; two should be plenty. + /* threadCount= */ 2, keepGoing, determineTests); this.targetParsingCompleteEvent = findPost(TargetParsingCompleteEvent.class); |