diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2019-04-12 22:52:58 +0200 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2019-04-12 22:52:58 +0200 |
commit | 8a4969c0db62ea95719c165639d872019a5f2e19 (patch) | |
tree | 01961b3336113623d8d57d75d0b58724dc6be035 /vespaclient-core/src/main/java/com/yahoo/clientmetrics/RouteMetricSet.java | |
parent | c59810dece3adb914554079621ecae191236cf1b (diff) |
Guard metrics and GC unused code
Diffstat (limited to 'vespaclient-core/src/main/java/com/yahoo/clientmetrics/RouteMetricSet.java')
-rw-r--r-- | vespaclient-core/src/main/java/com/yahoo/clientmetrics/RouteMetricSet.java | 19 |
1 files changed, 12 insertions, 7 deletions
diff --git a/vespaclient-core/src/main/java/com/yahoo/clientmetrics/RouteMetricSet.java b/vespaclient-core/src/main/java/com/yahoo/clientmetrics/RouteMetricSet.java index a0fab6dd6da..c5469e8a4f1 100644 --- a/vespaclient-core/src/main/java/com/yahoo/clientmetrics/RouteMetricSet.java +++ b/vespaclient-core/src/main/java/com/yahoo/clientmetrics/RouteMetricSet.java @@ -2,7 +2,9 @@ package com.yahoo.clientmetrics; import com.yahoo.messagebus.Reply; -import com.yahoo.metrics.*; +import com.yahoo.metrics.Metric; +import com.yahoo.metrics.MetricSet; +import com.yahoo.metrics.SumMetric; import java.util.HashMap; import java.util.Map; @@ -12,9 +14,9 @@ import java.util.Map; */ public class RouteMetricSet extends MetricSet { - SumMetric sum; - ProgressCallback callback; - Map<Integer,MessageTypeMetricSet> typeMap = new HashMap<Integer,MessageTypeMetricSet>(); + private final SumMetric sum; + private final ProgressCallback callback; + private final Map<Integer, MessageTypeMetricSet> typeMap = new HashMap<>(); public interface ProgressCallback { void onProgress(RouteMetricSet route); @@ -32,8 +34,10 @@ public class RouteMetricSet extends MetricSet { return "route"; } - public RouteMetricSet(RouteMetricSet source, CopyType copyType, MetricSet owner, boolean includeUnused) { + private RouteMetricSet(RouteMetricSet source, CopyType copyType, MetricSet owner, boolean includeUnused) { super(source, copyType, owner, includeUnused); + sum = null; + callback = null; } public void addReply(Reply r) { @@ -58,8 +62,9 @@ public class RouteMetricSet extends MetricSet { } @Override - public Metric clone(CopyType type, MetricSet owner, boolean includeUnused) - { return new RouteMetricSet(this, type, owner, includeUnused); } + public Metric clone(CopyType type, MetricSet owner, boolean includeUnused) { + return new RouteMetricSet(this, type, owner, includeUnused); + } String getRoute() { return getName(); |