diff options
author | gjoranv <gv@verizonmedia.com> | 2019-06-20 00:59:59 +0200 |
---|---|---|
committer | gjoranv <gv@verizonmedia.com> | 2019-06-20 00:59:59 +0200 |
commit | 987bd0c2d7128a043e3c0a88be791d53e1aeae12 (patch) | |
tree | 4d9048a7171a45937787f6f90a3d9f2487bb21ae | |
parent | ca937f838e0ed60d228aa43f1c3a3ba1e8376f9a (diff) |
Throw if there are two metric sets with the same id.
-rw-r--r-- | config-model/src/main/java/com/yahoo/vespa/model/admin/monitoring/builder/PredefinedMetricSets.java | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/admin/monitoring/builder/PredefinedMetricSets.java b/config-model/src/main/java/com/yahoo/vespa/model/admin/monitoring/builder/PredefinedMetricSets.java index 4638353f39f..d1d309bbd0b 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/admin/monitoring/builder/PredefinedMetricSets.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/admin/monitoring/builder/PredefinedMetricSets.java @@ -28,8 +28,11 @@ public class PredefinedMetricSets { private static Map<String, MetricSet> toMapById(MetricSet... metricSets) { Map<String, MetricSet> availableMetricSets = new LinkedHashMap<>(); - for (MetricSet metricSet : metricSets) - availableMetricSets.put(metricSet.getId(), metricSet); + for (MetricSet metricSet : metricSets) { + var existing = availableMetricSets.put(metricSet.getId(), metricSet); + if (existing != null) + throw new IllegalArgumentException("There are two predefined metric sets with id " + existing.getId()); + } return Collections.unmodifiableMap(availableMetricSets); } |