diff options
Diffstat (limited to 'configserver/src/main/java/com/yahoo/vespa/config/server/application/Application.java')
-rw-r--r-- | configserver/src/main/java/com/yahoo/vespa/config/server/application/Application.java | 28 |
1 files changed, 5 insertions, 23 deletions
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/application/Application.java b/configserver/src/main/java/com/yahoo/vespa/config/server/application/Application.java index 8e98bf5884b..baded957475 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/application/Application.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/application/Application.java @@ -105,16 +105,12 @@ public class Application implements ModelResult { defMd5 = ConfigUtils.getDefMd5(req.getDefContent().asList()); } ConfigCacheKey cacheKey = new ConfigCacheKey(configKey, defMd5); - if (logDebug()) { - debug("Resolving config " + cacheKey); - } + log.log(Level.FINE, () -> TenantRepository.logPre(getId()) + ("Resolving config " + cacheKey)); if (useCache(req)) { ConfigResponse config = cache.get(cacheKey); if (config != null) { - if (logDebug()) { - debug("Found config " + cacheKey + " in cache"); - } + log.log(Level.FINE, () -> TenantRepository.logPre(getId()) + ("Found config " + cacheKey + " in cache")); metricUpdater.incrementProcTime(System.currentTimeMillis() - start); return config; } @@ -125,9 +121,7 @@ public class Application implements ModelResult { metricUpdater.incrementFailedRequests(); throw new UnknownConfigDefinitionException("Unable to find config definition for '" + configKey.getNamespace() + "." + configKey.getName()); } - if (logDebug()) { - debug("Resolving " + configKey + " with config definition " + def); - } + log.log(Level.FINE, () -> TenantRepository.logPre(getId()) + ("Resolving " + configKey + " with config definition " + def)); ConfigInstance.Builder builder; ConfigPayload payload; @@ -174,27 +168,15 @@ public class Application implements ModelResult { return !request.noCache(); } - private boolean logDebug() { - return log.isLoggable(Level.FINE); - } - - private void debug(String message) { - log.log(Level.FINE, TenantRepository.logPre(getId()) + message); - } - private ConfigDefinition getTargetDef(GetConfigRequest req) { ConfigKey<?> configKey = req.getConfigKey(); DefContent def = req.getDefContent(); ConfigDefinitionKey configDefinitionKey = new ConfigDefinitionKey(configKey.getName(), configKey.getNamespace()); if (def.isEmpty()) { - if (logDebug()) { - debug("No config schema in request for " + configKey); - } + log.log(Level.FINE, () -> TenantRepository.logPre(getId()) + ("No config schema in request for " + configKey)); return cache.getDef(configDefinitionKey); } else { - if (logDebug()) { - debug("Got config schema from request, length:" + def.asList().size() + " : " + configKey); - } + log.log(Level.FINE, () -> TenantRepository.logPre(getId()) + ("Got config schema from request, length:" + def.asList().size() + " : " + configKey)); return new ConfigDefinition(configKey.getName(), def.asStringArray()); } } |