- Add trace as a field in the josn access log.

- Add the trace top the accesslog entry if debugging is enabled in SearchHandler.

1 files changed, 6 insertions, 3 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 0f0df3aa93e..dc066e65434 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
@@ -296,7 +296,10 @@ public class SearchHandler extends LoggingRequestHandler {
         // Transform result to response
         Renderer renderer = toRendererCopy(query.getPresentation().getRenderer());
         HttpSearchResponse response = new HttpSearchResponse(getHttpResponseStatus(request, result),
-                                                             result, query, renderer);
+                                                             result, query, renderer,
+                                                             log.isLoggable(Level.FINE)
+                                                                     ? query.getContext(false).getTrace().traceNode()
+                                                                     : null);
         if (hostResponseHeaderKey.isPresent())
             response.headers().add(hostResponseHeaderKey.get(), selfHostname);
 
@@ -410,7 +413,7 @@ public class SearchHandler extends LoggingRequestHandler {
         } catch (ParseException e) {
             ErrorMessage error = ErrorMessage.createIllegalQuery("Could not parse query [" + request + "]: "
                                                                  + Exceptions.toMessageString(e));
-            log.log(LogLevel.DEBUG, () -> error.getDetailedMessage());
+            log.log(LogLevel.DEBUG, error.getDetailedMessage());
             return new Result(query, error);
         } catch (IllegalArgumentException e) {
             if ("Comparison method violates its general contract!".equals(e.getMessage())) {
@@ -422,7 +425,7 @@ public class SearchHandler extends LoggingRequestHandler {
             else {
                 ErrorMessage error = ErrorMessage.createBadRequest("Invalid search request [" + request + "]: "
                                                                    + Exceptions.toMessageString(e));
-                log.log(LogLevel.DEBUG, () -> error.getDetailedMessage());
+                log.log(LogLevel.DEBUG, error.getDetailedMessage());
                 return new Result(query, error);
             }
         } catch (LinkageError | StackOverflowError e) {