diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2021-10-09 21:18:42 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-10-09 21:18:42 +0200 |
commit | 165142127b8d40621fdd29a2fcac74084eeb4d9c (patch) | |
tree | 345f741099faf0525143da30477772fe4bbc2835 /container-search/src/main/java | |
parent | ad390fb973964e65348465d14af57827a7703bd0 (diff) | |
parent | f8f44b663a2a5ef6cbe439773ff61f2a32a4a65c (diff) |
Merge pull request #19489 from vespa-engine/balder/avoid-creating-unused-singlethreaded-executor-that-is-discarded-just-afterwards
No need to create a temporary execution, that is discarded right afte…
Diffstat (limited to 'container-search/src/main/java')
-rw-r--r-- | container-search/src/main/java/com/yahoo/search/searchchain/Execution.java | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/container-search/src/main/java/com/yahoo/search/searchchain/Execution.java b/container-search/src/main/java/com/yahoo/search/searchchain/Execution.java index 15253f154ac..aaf74cfd355 100644 --- a/container-search/src/main/java/com/yahoo/search/searchchain/Execution.java +++ b/container-search/src/main/java/com/yahoo/search/searchchain/Execution.java @@ -123,7 +123,7 @@ public class Execution extends com.yahoo.processing.execution.Execution { this.tokenRegistry = tokenRegistry; this.rendererRegistry = rendererRegistry; this.linguistics = linguistics; - this.executor = executor != null ? executor : Executors.newSingleThreadExecutor(); + this.executor = executor; } /** @deprecated pass an executor */ @@ -135,7 +135,7 @@ public class Execution extends com.yahoo.processing.execution.Execution { /** Creates a context stub with no information. This is for unit testing. */ public static Context createContextStub() { - return new Context(null, null, null, null, null, null); + return createContextStub(null); } /** @@ -143,7 +143,7 @@ public class Execution extends com.yahoo.processing.execution.Execution { * initialized. This is for unit testing. */ public static Context createContextStub(IndexFacts indexFacts) { - return new Context(null, indexFacts, null, null, null, null); + return createContextStub(null, indexFacts); } /** @@ -151,7 +151,7 @@ public class Execution extends com.yahoo.processing.execution.Execution { * initialized. This is for unit testing. */ public static Context createContextStub(SearchChainRegistry searchChainRegistry, IndexFacts indexFacts) { - return new Context(searchChainRegistry, indexFacts, null, null, null, null); + return createContextStub(searchChainRegistry, indexFacts, null); } /** @@ -159,7 +159,7 @@ public class Execution extends com.yahoo.processing.execution.Execution { * initialized. This is for unit testing. */ public static Context createContextStub(SearchChainRegistry searchChainRegistry, IndexFacts indexFacts, Linguistics linguistics) { - return new Context(searchChainRegistry, indexFacts, null, null, linguistics, null); + return new Context(searchChainRegistry, indexFacts, null, null, linguistics, Executors.newSingleThreadExecutor()); } /** |