diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2023-03-10 14:10:55 +0100 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2023-03-10 14:10:55 +0100 |
commit | ce679911fd827f5070ebb024af954dd889938b52 (patch) | |
tree | b8b5044dc82c7c73942fc1948a5e3866cefcf06c /container-core/src/main/java/com/yahoo/container | |
parent | b9094db1b54c567abcc107e518c119858a622f0c (diff) |
Avoid getting query before it is necessary.
Diffstat (limited to 'container-core/src/main/java/com/yahoo/container')
-rw-r--r-- | container-core/src/main/java/com/yahoo/container/jdisc/LoggingRequestHandler.java | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/container-core/src/main/java/com/yahoo/container/jdisc/LoggingRequestHandler.java b/container-core/src/main/java/com/yahoo/container/jdisc/LoggingRequestHandler.java index 4c287deb82f..bacceac9d76 100644 --- a/container-core/src/main/java/com/yahoo/container/jdisc/LoggingRequestHandler.java +++ b/container-core/src/main/java/com/yahoo/container/jdisc/LoggingRequestHandler.java @@ -101,7 +101,7 @@ public abstract class LoggingRequestHandler extends ThreadedHttpRequestHandler { private void logTimes(long startTime, String sourceIP, long renderStartTime, long commitStartTime, long endTime, - String req, String normalizedQuery, Timing t) { + String req, ExtendedResponse response) { // note: intentionally only taking time since request was received long totalTime = endTime - startTime; @@ -109,6 +109,7 @@ public abstract class LoggingRequestHandler extends ThreadedHttpRequestHandler { long timeoutInterval; long requestOverhead; long summaryStartTime; + Timing t = response.getTiming(); if (t != null) { timeoutInterval = t.getTimeout(); long queryStartTime = t.getQueryStartTime(); @@ -124,7 +125,7 @@ public abstract class LoggingRequestHandler extends ThreadedHttpRequestHandler { log.log(Level.FINE, () -> { StringBuilder b = new StringBuilder(); - b.append(normalizedQuery); + b.append(response.getParsedQuery()); b.append(" from ").append(sourceIP).append(". "); if (requestOverhead > 0) { b.append("Time from HTTP connection open to request reception "); @@ -222,8 +223,7 @@ public abstract class LoggingRequestHandler extends ThreadedHttpRequestHandler { commitStartTime, endTime, getUri(jdiscRequest), - extendedResponse.getParsedQuery(), - extendedResponse.getTiming()); + extendedResponse); Optional<AccessLogEntry> jdiscRequestAccessLogEntry = AccessLoggingRequestHandler.getAccessLogEntry(jdiscRequest); |