1 files changed, 5 insertions, 15 deletions
diff --git a/metrics-proxy/src/main/java/ai/vespa/metricsproxy/metric/model/MetricsPacket.java b/metrics-proxy/src/main/java/ai/vespa/metricsproxy/metric/model/MetricsPacket.java
index 02ccd2c988c..fff2302f98b 100644
--- a/metrics-proxy/src/main/java/ai/vespa/metricsproxy/metric/model/MetricsPacket.java
+++ b/metrics-proxy/src/main/java/ai/vespa/metricsproxy/metric/model/MetricsPacket.java
@@ -4,7 +4,6 @@ package ai.vespa.metricsproxy.metric.model;
 import ai.vespa.metricsproxy.metric.Metric;
 
 import java.time.Instant;
-import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Collections;
 import java.util.LinkedHashMap;
@@ -15,7 +14,6 @@ import java.util.Objects;
 import java.util.Set;
 import java.util.function.Function;
 
-import static java.util.Collections.unmodifiableList;
 import static java.util.Collections.unmodifiableMap;
 import static java.util.stream.Collectors.joining;
 
@@ -44,22 +42,14 @@ public class MetricsPacket {
         this.statusMessage = statusMessage;
         this.timestamp = timestamp;
         this.service = service;
-        this.metrics = metrics;
-        this.dimensions = dimensions;
+        this.metrics = Map.copyOf(metrics);
+        this.dimensions = unmodifiableMap(dimensions);
         this.consumers = Set.copyOf(consumers);
     }
 
-    public Map<MetricId, Number> metrics() {
-        return unmodifiableMap(metrics);
-    }
-
-    public Map<DimensionId, String> dimensions() {
-        return unmodifiableMap(dimensions);
-    }
-
-    public Set<ConsumerId> consumers() {
-        return consumers;
-    }
+    public Map<MetricId, Number> metrics() { return metrics; }
+    public Map<DimensionId, String> dimensions() { return dimensions; }
+    public Set<ConsumerId> consumers() { return consumers;}
 
     @Override
     public String toString() {