diff options
author | Harald Musum <musum@yahooinc.com> | 2022-01-24 14:05:02 +0100 |
---|---|---|
committer | Harald Musum <musum@yahooinc.com> | 2022-01-24 14:05:02 +0100 |
commit | 541522c51feeeb4f0cb655a167f0c473482cdf47 (patch) | |
tree | 71efa14e484e2b93eb0289354f1f70c8a6e7e03c /config | |
parent | 8e918835acb36f033ce6c9a087b1337ccf5227f1 (diff) |
Add more info in log message when config unexpectedly changes
Diffstat (limited to 'config')
-rw-r--r-- | config/src/main/java/com/yahoo/config/subscription/impl/ConfigSubscription.java | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/config/src/main/java/com/yahoo/config/subscription/impl/ConfigSubscription.java b/config/src/main/java/com/yahoo/config/subscription/impl/ConfigSubscription.java index f8a45a11b70..a9587da0f0b 100644 --- a/config/src/main/java/com/yahoo/config/subscription/impl/ConfigSubscription.java +++ b/config/src/main/java/com/yahoo/config/subscription/impl/ConfigSubscription.java @@ -200,11 +200,9 @@ public abstract class ConfigSubscription<T extends ConfigInstance> { void setConfigAndGeneration(Long generation, boolean applyOnRestart, T config, PayloadChecksums payloadChecksums) { ConfigState<T> prev = this.config.get(); boolean configChanged = !Objects.equals(prev.getConfig(), config); - String message = "Config has changed unexpectedly for " + key + ", generation " + generation; if (configChanged) { - if (log.isLoggable(Level.FINE)) - message = message + ", config in state :" + prev.getConfig() + ", new config: " + config; - log.log(Level.WARNING, message); + log.log(Level.WARNING, "Config has changed unexpectedly for " + key + ", generation " + generation + + ", config in state :" + prev.getConfig() + ", new config: " + config); } this.config.set(new ConfigState<>(true, generation, applyOnRestart, configChanged, config, payloadChecksums)); } |