Merge pull request #4237 from vespa-engine/hmusum/log-when-config-is-updated

Log when config is updated

1 files changed, 7 insertions, 4 deletions

diff --git a/config/src/vespa/config/subscription/configsubscriptionset.cpp b/config/src/vespa/config/subscription/configsubscriptionset.cpp
index 51c29823664..e82262d2d3c 100644
--- a/config/src/vespa/config/subscription/configsubscriptionset.cpp
+++ b/config/src/vespa/config/subscription/configsubscriptionset.cpp
@@ -91,10 +91,13 @@ ConfigSubscriptionSet::acquireSnapshot(uint64_t timeoutInMillis, bool ignoreChan
         LOG(spam, "Config was updated from %ld to %ld", _currentGeneration, lastGeneration);
         _currentGeneration = lastGeneration;
         _state = CONFIGURED;
-        for (SubscriptionList::iterator it(_subscriptionList.begin()), mt(_subscriptionList.end());
-             it != mt;
-             it++) {
-            (*it)->flip();
+        for (const auto & subscription : _subscriptionList) {
+            const ConfigKey & key(subscription->getKey());
+            LOG(debug, "Updated config id(%s), has changed: %s, lastGenerationChanged: %ld",
+                key.getConfigId().c_str(),
+                (subscription->hasChanged() ? "true" : "false"),
+                subscription->getLastGenerationChanged());
+            subscription->flip();
         }
     }
     return updated;