From ab3fb1709039a318361f3078d878448a0f25ced1 Mon Sep 17 00:00:00 2001
From: Arne Juul <arnej@yahoo-inc.com>
Date: Wed, 3 Oct 2018 16:56:54 +0000
Subject: totally remove dimension key concept

---
 metrics/src/vespa/metrics/jsonwriter.cpp | 18 +-----------------
 metrics/src/vespa/metrics/jsonwriter.h   |  2 --
 metrics/src/vespa/metrics/metricset.cpp  | 16 ++++------------
 metrics/src/vespa/metrics/metricset.h    | 11 +----------
 4 files changed, 6 insertions(+), 41 deletions(-)

(limited to 'metrics')

diff --git a/metrics/src/vespa/metrics/jsonwriter.cpp b/metrics/src/vespa/metrics/jsonwriter.cpp
index 8dd5e8fdbcb..6ea585f0635 100644
--- a/metrics/src/vespa/metrics/jsonwriter.cpp
+++ b/metrics/src/vespa/metrics/jsonwriter.cpp
@@ -41,26 +41,10 @@ JsonWriter::doneVisitingSnapshot(const MetricSnapshot&)
     _period = 0;
 }
 
-void
-JsonWriter::pushLegacyDimensionFromSet(const MetricSet& ms)
-{
-    _dimensionStack.push_back({{ms.getDimensionKey(), ms.getName()}});
-}
-
-void
-JsonWriter::pushAllDimensionsFromSet(const MetricSet& ms)
-{
-    _dimensionStack.push_back(ms.getTags());
-}
-
 bool
 JsonWriter::visitMetricSet(const MetricSet& set, bool)
 {
-    if (!set.getDimensionKey().empty()) {
-        pushLegacyDimensionFromSet(set);
-    } else {
-        pushAllDimensionsFromSet(set);
-    }
+    _dimensionStack.push_back(set.getTags());
     return true;
 }
 
diff --git a/metrics/src/vespa/metrics/jsonwriter.h b/metrics/src/vespa/metrics/jsonwriter.h
index 07fdc9c22c2..300f0cbb0ec 100644
--- a/metrics/src/vespa/metrics/jsonwriter.h
+++ b/metrics/src/vespa/metrics/jsonwriter.h
@@ -33,8 +33,6 @@ private:
     void writeCommonPrefix(const Metric& m);
     void writeCommonPostfix(const Metric& m);
 
-    void pushLegacyDimensionFromSet(const MetricSet&);
-    void pushAllDimensionsFromSet(const MetricSet&);
     void writeDimensions(const DimensionSet&);
     void writeInheritedDimensions();
     void writeMetricSpecificDimensions(const Metric&);
diff --git a/metrics/src/vespa/metrics/metricset.cpp b/metrics/src/vespa/metrics/metricset.cpp
index 2e708e17504..cfaeb8b6f02 100644
--- a/metrics/src/vespa/metrics/metricset.cpp
+++ b/metrics/src/vespa/metrics/metricset.cpp
@@ -15,12 +15,10 @@ LOG_SETUP(".metrics.metricsset");
 namespace metrics {
 
 MetricSet::MetricSet(const String& name, const String& tags,
-                     const String& description, MetricSet* owner,
-                     const std::string& dimensionKey)
+                     const String& description, MetricSet* owner)
     : Metric(name, tags, description, owner),
       _metricOrder(),
-      _registrationAltered(false),
-      _dimensionKey(dimensionKey)
+      _registrationAltered(false)
 {
 }
 
@@ -28,8 +26,7 @@ MetricSet::MetricSet(const String& name, Tags dimensions,
                      const String& description, MetricSet* owner)
     : Metric(name, std::move(dimensions), description, owner),
       _metricOrder(),
-      _registrationAltered(false),
-      _dimensionKey()
+      _registrationAltered(false)
 {
 }
 
@@ -40,13 +37,8 @@ MetricSet::MetricSet(const MetricSet& other,
                      bool includeUnused)
     : Metric(other, owner),
       _metricOrder(),
-      _registrationAltered(false),
-      _dimensionKey(other._dimensionKey)
+      _registrationAltered(false)
 {
-    if (copyType == INACTIVE && owner == NULL && includeUnused) {
-        _dimensionKey = "";
-    }
-
     for (const Metric* metric : other._metricOrder) {
         if (copyType != INACTIVE || includeUnused || metric->used()) {
             Metric* m = metric->clone(ownerList, copyType, this, includeUnused);
diff --git a/metrics/src/vespa/metrics/metricset.h b/metrics/src/vespa/metrics/metricset.h
index 9671aaffbfc..48027db374a 100644
--- a/metrics/src/vespa/metrics/metricset.h
+++ b/metrics/src/vespa/metrics/metricset.h
@@ -20,14 +20,10 @@ class MetricSet : public Metric
     bool _registrationAltered; // Set to true if metrics have been
                                // registered/unregistered since last time
                                // it was reset
-    std::string _dimensionKey; // If this metric is part of a monitoring dimension,
-                               // the key of the dimension should be set here.
-                               // If so, the name of the metric is used as dimension value.
 
 public:
     MetricSet(const String& name, const String& tags,
-              const String& description, MetricSet* owner = 0,
-              const std::string& dimensionKey = "");
+              const String& description, MetricSet* owner = 0);
 
     MetricSet(const String& name, Tags dimensions,
               const String& description, MetricSet* owner = 0);
@@ -85,11 +81,6 @@ public:
     bool isMetricSet() const override { return true; }
     void addToPart(Metric& m) const override { addTo(m, 0); }
 
-    /**
-     * Returns the key of the dimension this metric is part of (if any).
-     */
-    const std::string& getDimensionKey() const { return _dimensionKey; }
-
 private:
         // Do not generate default copy constructor or assignment operator
         // These would screw up metric registering
-- 
cgit v1.2.3