diff options
| author | Bjørn Christian Seime <bjorncs@verizonmedia.com> | 2021-01-21 15:49:05 +0100 |
|---|---|---|
| committer | Bjørn Christian Seime <bjorncs@verizonmedia.com> | 2021-01-21 16:54:09 +0100 |
| commit | 6cc469b7950f4f6f2b67596b162cd73e59c22336 (patch) | |
| tree | b19e9a8bf8c1bda395da8862494c3edafabf9b4e /container-search | |
| parent | be7847731521fb516269b65905029c1639d4aec8 (diff) | |
Deprecate LoggingRequestHandler constructors taking AccessLog
Add replacement constructors without AccessLog.
Remove use of deprecated constructors for internal handlers.
Diffstat (limited to 'container-search')
| -rw-r--r-- | container-search/src/main/java/com/yahoo/search/handler/SearchHandler.java | 17 |
1 files changed, 7 insertions, 10 deletions
diff --git a/container-search/src/main/java/com/yahoo/search/handler/SearchHandler.java b/container-search/src/main/java/com/yahoo/search/handler/SearchHandler.java index 0337fe9eab9..2b27f60ef73 100644 --- a/container-search/src/main/java/com/yahoo/search/handler/SearchHandler.java +++ b/container-search/src/main/java/com/yahoo/search/handler/SearchHandler.java @@ -120,24 +120,23 @@ public class SearchHandler extends LoggingRequestHandler { public SearchHandler(Statistics statistics, Metric metric, ContainerThreadPool threadpool, - AccessLog accessLog, + AccessLog ignored, CompiledQueryProfileRegistry queryProfileRegistry, ContainerHttpConfig config, ExecutionFactory executionFactory) { - this(statistics, metric, threadpool.executor(), accessLog, queryProfileRegistry, config, executionFactory); + this(statistics, metric, threadpool.executor(), ignored, queryProfileRegistry, config, executionFactory); } public SearchHandler(Statistics statistics, Metric metric, Executor executor, - AccessLog accessLog, + AccessLog ignored, CompiledQueryProfileRegistry queryProfileRegistry, ContainerHttpConfig containerHttpConfig, ExecutionFactory executionFactory) { this(statistics, metric, executor, - accessLog, queryProfileRegistry, executionFactory, containerHttpConfig.numQueriesToTraceOnDebugAfterConstruction(), @@ -152,14 +151,13 @@ public class SearchHandler extends LoggingRequestHandler { public SearchHandler(Statistics statistics, Metric metric, Executor executor, - AccessLog accessLog, + AccessLog ignored, QueryProfilesConfig queryProfileConfig, ContainerHttpConfig containerHttpConfig, ExecutionFactory executionFactory) { this(statistics, metric, executor, - accessLog, QueryProfileConfigurer.createFromConfig(queryProfileConfig).compile(), executionFactory, containerHttpConfig.numQueriesToTraceOnDebugAfterConstruction(), @@ -170,22 +168,21 @@ public class SearchHandler extends LoggingRequestHandler { public SearchHandler(Statistics statistics, Metric metric, Executor executor, - AccessLog accessLog, + AccessLog ignored, CompiledQueryProfileRegistry queryProfileRegistry, ExecutionFactory executionFactory, Optional<String> hostResponseHeaderKey) { - this(statistics, metric, executor, accessLog, queryProfileRegistry, executionFactory, 0, hostResponseHeaderKey); + this(statistics, metric, executor, queryProfileRegistry, executionFactory, 0, hostResponseHeaderKey); } private SearchHandler(Statistics statistics, Metric metric, Executor executor, - AccessLog accessLog, CompiledQueryProfileRegistry queryProfileRegistry, ExecutionFactory executionFactory, long numQueriesToTraceOnDebugAfterStartup, Optional<String> hostResponseHeaderKey) { - super(executor, accessLog, metric, true); + super(executor, metric, true); log.log(Level.FINE, "SearchHandler.init " + System.identityHashCode(this)); this.queryProfileRegistry = queryProfileRegistry; this.executionFactory = executionFactory; |