re-compute mangled name

* must re-compute mangled name whenever name or tags change,
  including when tags are set to empty set.

2 files changed, 7 insertions, 1 deletions

diff --git a/metrics/src/vespa/metrics/metric.cpp b/metrics/src/vespa/metrics/metric.cpp
index f6be5f79357..e67398e4626 100644
--- a/metrics/src/vespa/metrics/metric.cpp
+++ b/metrics/src/vespa/metrics/metric.cpp
@@ -116,6 +116,7 @@ void
 Metric::assignMangledNameWithDimensions()
 {
     if (!tagsSpecifyAtLeastOneDimension(_tags)) {
+        _mangledName = _name;
         return;
     }
     sortTagsInDeterministicOrder();
diff --git a/metrics/src/vespa/metrics/metric.h b/metrics/src/vespa/metrics/metric.h
index 5b617e68a7f..91ad88fecf7 100644
--- a/metrics/src/vespa/metrics/metric.h
+++ b/metrics/src/vespa/metrics/metric.h
@@ -215,12 +215,17 @@ public:
     virtual bool visit(MetricVisitor& visitor, bool tagAsAutoGenerated = false) const = 0;
 
     /** Used by sum metric to alter name of cloned metric for sum. */
-    void setName(const String& name) { _name = name; }
+    void setName(const String& name) {
+        _name = name;
+        assignMangledNameWithDimensions();
+    }
+
     /** Used by sum metric to alter description of cloned metric for sum. */
     void setDescription(const String& d) { _description = d; }
     /** Used by sum metric to alter tag of cloned metric for sum. */
     void setTags(Tags tags) {
         _tags = std::move(tags);
+        assignMangledNameWithDimensions(); 
     }
 
     /** Set whether metrics have ever been set. */