feat: deprecate fragmented metric metadata API

zeitlinger · zeitlinger · commit 1dffd50a4404 · 2026-05-18T15:21:58.000Z
Signed-off-by: Gregor Zeitlinger &lt;gregor.zeitlinger@grafana.com&gt;
diff --git a/examples/example-exporter-multi-target/src/main/java/io/prometheus/metrics/examples/multitarget/SampleMultiCollector.java b/examples/example-exporter-multi-target/src/main/java/io/prometheus/metrics/examples/multitarget/SampleMultiCollector.java
@@ -78,6 +78,8 @@ protected MetricSnapshots collectMetricSnapshots(PrometheusScrapeRequest scrapeR
   }
 
   @Override
+  @Deprecated
+  @SuppressWarnings("InlineMeSuggester")
   public List<String> getPrometheusNames() {
     List<String> names = new ArrayList<String>();
     names.add("x_calls_total");
diff --git a/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/Counter.java b/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/Counter.java
@@ -90,10 +90,12 @@ protected CounterSnapshot collect(List<Labels> labels, List<DataPoint> metricDat
     for (int i = 0; i < labels.size(); i++) {
       data.add(metricData.get(i).collect(labels.get(i)));
     }
-    return new CounterSnapshot(getMetadata(), data);
+    return new CounterSnapshot(metadata, data);
   }
 
   @Override
+  @Deprecated
+  @SuppressWarnings("InlineMeSuggester")
   public MetricType getMetricType() {
     return MetricType.COUNTER;
   }
diff --git a/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/CounterWithCallback.java b/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/CounterWithCallback.java
@@ -48,10 +48,12 @@ public CounterSnapshot collect() {
               new CounterSnapshot.CounterDataPointSnapshot(
                   value, makeLabels(labelValues), null, 0L));
         });
-    return new CounterSnapshot(getMetadata(), dataPoints);
+    return new CounterSnapshot(metadata, dataPoints);
   }
 
   @Override
+  @Deprecated
+  @SuppressWarnings("InlineMeSuggester")
   public MetricType getMetricType() {
     return MetricType.COUNTER;
   }
diff --git a/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/Gauge.java b/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/Gauge.java
@@ -92,10 +92,12 @@ protected GaugeSnapshot collect(List<Labels> labels, List<DataPoint> metricData)
     for (int i = 0; i < labels.size(); i++) {
       dataPointSnapshots.add(metricData.get(i).collect(labels.get(i)));
     }
-    return new GaugeSnapshot(getMetadata(), dataPointSnapshots);
+    return new GaugeSnapshot(metadata, dataPointSnapshots);
   }
 
   @Override
+  @Deprecated
+  @SuppressWarnings("InlineMeSuggester")
   public MetricType getMetricType() {
     return MetricType.GAUGE;
   }
diff --git a/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/GaugeWithCallback.java b/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/GaugeWithCallback.java
@@ -52,10 +52,12 @@ public GaugeSnapshot collect() {
           dataPoints.add(
               new GaugeSnapshot.GaugeDataPointSnapshot(value, makeLabels(labelValues), null, 0L));
         });
-    return new GaugeSnapshot(getMetadata(), dataPoints);
+    return new GaugeSnapshot(metadata, dataPoints);
   }
 
   @Override
+  @Deprecated
+  @SuppressWarnings("InlineMeSuggester")
   public MetricType getMetricType() {
     return MetricType.GAUGE;
   }
diff --git a/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/Histogram.java b/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/Histogram.java
@@ -647,10 +647,12 @@ protected HistogramSnapshot collect(List<Labels> labels, List<DataPoint> metricD
     for (int i = 0; i < labels.size(); i++) {
       data.add(metricData.get(i).collect(labels.get(i)));
     }
-    return new HistogramSnapshot(getMetadata(), data);
+    return new HistogramSnapshot(metadata, data);
   }
 
   @Override
+  @Deprecated
+  @SuppressWarnings("InlineMeSuggester")
   public MetricType getMetricType() {
     return MetricType.HISTOGRAM;
   }
diff --git a/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/Info.java b/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/Info.java
@@ -48,7 +48,7 @@ public void setLabelValues(String... labelValues) {
       throw new IllegalArgumentException(
           getClass().getSimpleName()
               + " "
-              + getMetadata().getName()
+              + metadata.getName()
               + " was created with "
               + labelNames.length
               + " label names, but you called setLabelValues() with "
@@ -66,7 +66,7 @@ public void addLabelValues(String... labelValues) {
       throw new IllegalArgumentException(
           getClass().getSimpleName()
               + " "
-              + getMetadata().getName()
+              + metadata.getName()
               + " was created with "
               + labelNames.length
               + " label names, but you called addLabelValues() with "
@@ -82,7 +82,7 @@ public void remove(String... labelValues) {
       throw new IllegalArgumentException(
           getClass().getSimpleName()
               + " "
-              + getMetadata().getName()
+              + metadata.getName()
               + " was created with "
               + labelNames.length
               + " label names, but you called remove() with "
@@ -103,10 +103,12 @@ public InfoSnapshot collect() {
         data.add(new InfoSnapshot.InfoDataPointSnapshot(label.merge(constLabels)));
       }
     }
-    return new InfoSnapshot(getMetadata(), data);
+    return new InfoSnapshot(metadata, data);
   }
 
   @Override
+  @Deprecated
+  @SuppressWarnings("InlineMeSuggester")
   public MetricType getMetricType() {
     return MetricType.INFO;
   }
diff --git a/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/MetricWithFixedMetadata.java b/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/MetricWithFixedMetadata.java
@@ -1,8 +1,10 @@
 package io.prometheus.metrics.core.metrics;
 
 import io.prometheus.metrics.config.PrometheusProperties;
+import io.prometheus.metrics.model.registry.MetricType;
 import io.prometheus.metrics.model.snapshots.Label;
 import io.prometheus.metrics.model.snapshots.Labels;
+import io.prometheus.metrics.model.snapshots.MetricFamilyDescriptor;
 import io.prometheus.metrics.model.snapshots.MetricMetadata;
 import io.prometheus.metrics.model.snapshots.PrometheusNaming;
 import io.prometheus.metrics.model.snapshots.Unit;
@@ -20,7 +22,7 @@
  */
 public abstract class MetricWithFixedMetadata extends Metric {
 
-  private final MetricMetadata metadata;
+  protected final MetricMetadata metadata;
   protected final String[] labelNames;
 
   protected MetricWithFixedMetadata(Builder<?, ?> builder) {
@@ -37,6 +39,19 @@ protected MetricWithFixedMetadata(Builder<?, ?> builder) {
   }
 
   @Override
+  @Nullable
+  @SuppressWarnings("deprecation")
+  public MetricFamilyDescriptor getMetricFamilyDescriptor() {
+    MetricType metricType = getMetricType();
+    if (metricType == null) {
+      return null;
+    }
+    return MetricFamilyDescriptor.of(metricType, metadata, getPrometheusLabels());
+  }
+
+  @Override
+  @Deprecated
+  @SuppressWarnings("InlineMeSuggester")
   public MetricMetadata getMetadata() {
     return metadata;
   }
@@ -66,12 +81,20 @@ private String makeExpositionBaseName(@Nullable String expositionBaseName, @Null
   }
 
   @Override
+  @Deprecated
+  @SuppressWarnings("InlineMeSuggester")
   public String getPrometheusName() {
     return metadata.getPrometheusName();
   }
 
   @Override
+  @Deprecated
+  @SuppressWarnings("InlineMeSuggester")
   public Set<String> getLabelNames() {
+    return getPrometheusLabels();
+  }
+
+  private Set<String> getPrometheusLabels() {
     Set<String> names = new HashSet<>();
     for (String labelName : labelNames) {
       names.add(PrometheusNaming.prometheusName(labelName));
diff --git a/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/StateSet.java b/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/StateSet.java
@@ -60,7 +60,7 @@ private StateSet(Builder builder, String[] names) {
     super(builder);
     this.names = names;
     for (String name : names) {
-      if (this.getMetadata().getPrometheusName().equals(prometheusName(name))) {
+      if (metadata.getPrometheusName().equals(prometheusName(name))) {
         throw new IllegalArgumentException(
             "Label name "
                 + name
@@ -82,10 +82,12 @@ protected StateSetSnapshot collect(List<Labels> labels, List<DataPoint> metricDa
           new StateSetSnapshot.StateSetDataPointSnapshot(
               names, metricDataList.get(i).values, labels.get(i)));
     }
-    return new StateSetSnapshot(getMetadata(), data);
+    return new StateSetSnapshot(metadata, data);
   }
 
   @Override
+  @Deprecated
+  @SuppressWarnings("InlineMeSuggester")
   public MetricType getMetricType() {
     return MetricType.STATESET;
   }
diff --git a/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/StatefulMetric.java b/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/StatefulMetric.java
@@ -105,7 +105,7 @@ public D labelValues(String... labelValues) {
         throw new IllegalArgumentException(
             getClass().getSimpleName()
                 + " "
-                + getMetadata().getName()
+                + metadata.getName()
                 + " was created with label names, so you must call labelValues(...)"
                 + " when using it.");
       } else {
@@ -120,7 +120,7 @@ public D labelValues(String... labelValues) {
             if (l.get(i) == null) {
               throw new IllegalArgumentException(
                   "null label value for metric "
-                      + getMetadata().getName()
+                      + metadata.getName()
                       + " and label "
                       + labelNames[i]);
             }
@@ -171,7 +171,7 @@ protected MetricsProperties[] getMetricProperties(
     if (Objects.equals(builder.exemplarsEnabled, false)) {
       properties.add(MetricsProperties.builder().exemplarsEnabled(false).build());
     }
-    String metricName = getMetadata().getName();
+    String metricName = metadata.getName();
     if (prometheusProperties.getMetricProperties(metricName) != null) {
       properties.add(prometheusProperties.getMetricProperties(metricName));
     }
diff --git a/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/Summary.java b/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/Summary.java
@@ -116,10 +116,12 @@ protected SummarySnapshot collect(List<Labels> labels, List<DataPoint> metricDat
     for (int i = 0; i < labels.size(); i++) {
       data.add(metricData.get(i).collect(labels.get(i)));
     }
-    return new SummarySnapshot(getMetadata(), data);
+    return new SummarySnapshot(metadata, data);
   }
 
   @Override
+  @Deprecated
+  @SuppressWarnings("InlineMeSuggester")
   public MetricType getMetricType() {
     return MetricType.SUMMARY;
   }
diff --git a/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/SummaryWithCallback.java b/prometheus-metrics-core/src/main/java/io/prometheus/metrics/core/metrics/SummaryWithCallback.java
@@ -61,10 +61,12 @@ public SummarySnapshot collect() {
               new SummarySnapshot.SummaryDataPointSnapshot(
                   count, sum, quantiles, makeLabels(labelValues), Exemplars.EMPTY, 0L));
         });
-    return new SummarySnapshot(getMetadata(), dataPoints);
+    return new SummarySnapshot(metadata, dataPoints);
   }
 
   @Override
+  @Deprecated
+  @SuppressWarnings("InlineMeSuggester")
   public MetricType getMetricType() {
     return MetricType.SUMMARY;
   }
diff --git a/prometheus-metrics-core/src/test/java/io/prometheus/metrics/core/metrics/MetricWithFixedMetadataTest.java b/prometheus-metrics-core/src/test/java/io/prometheus/metrics/core/metrics/MetricWithFixedMetadataTest.java
@@ -0,0 +1,72 @@
+package io.prometheus.metrics.core.metrics;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+import io.prometheus.metrics.config.PrometheusProperties;
+import io.prometheus.metrics.model.registry.MetricType;
+import io.prometheus.metrics.model.snapshots.Labels;
+import io.prometheus.metrics.model.snapshots.MetricFamilyDescriptor;
+import io.prometheus.metrics.model.snapshots.MetricSnapshot;
+import java.util.Collections;
+import org.junit.jupiter.api.Test;
+
+class MetricWithFixedMetadataTest {
+
+  @Test
+  @SuppressWarnings("deprecation")
+  void getMetricFamilyDescriptorAdaptsDeprecatedMetricTypeOverride() {
+    LegacyMetric metric =
+        LegacyMetric.builder()
+            .name("legacy.metric")
+            .constLabels(Labels.of("const.label", "value"))
+            .labelNames("dynamic.label")
+            .build();
+
+    MetricFamilyDescriptor descriptor = metric.getMetricFamilyDescriptor();
+
+    assertThat(descriptor).isNotNull();
+    assertThat(descriptor.getType()).isEqualTo(MetricType.GAUGE);
+    assertThat(descriptor.getPrometheusName()).isEqualTo("legacy_metric");
+    assertThat(descriptor.getLabelNames())
+        .containsExactlyInAnyOrder("const_label", "dynamic_label");
+  }
+
+  private static class LegacyMetric extends MetricWithFixedMetadata {
+
+    private LegacyMetric(Builder builder) {
+      super(builder);
+    }
+
+    private static Builder builder() {
+      return new Builder();
+    }
+
+    @Override
+    public MetricSnapshot collect() {
+      throw new UnsupportedOperationException();
+    }
+
+    @Override
+    @Deprecated
+    public MetricType getMetricType() {
+      return MetricType.GAUGE;
+    }
+
+    private static class Builder extends MetricWithFixedMetadata.Builder<Builder, LegacyMetric> {
+
+      private Builder() {
+        super(Collections.emptyList(), PrometheusProperties.builder().build());
+      }
+
+      @Override
+      public LegacyMetric build() {
+        return new LegacyMetric(this);
+      }
+
+      @Override
+      protected Builder self() {
+        return this;
+      }
+    }
+  }
+}
diff --git a/prometheus-metrics-exposition-formats/src/test/java/io/prometheus/metrics/expositionformats/DuplicateNamesProtobufTest.java b/prometheus-metrics-exposition-formats/src/test/java/io/prometheus/metrics/expositionformats/DuplicateNamesProtobufTest.java
@@ -19,7 +19,7 @@
 import java.util.List;
 import org.junit.jupiter.api.Test;
 
-@SuppressWarnings("NonCanonicalType")
+@SuppressWarnings({"NonCanonicalType", "deprecation"})
 class DuplicateNamesProtobufTest {
 
   private static PrometheusRegistry getPrometheusRegistry() {
diff --git a/prometheus-metrics-exposition-textformats/src/test/java/io/prometheus/metrics/expositionformats/DuplicateNamesExpositionTest.java b/prometheus-metrics-exposition-textformats/src/test/java/io/prometheus/metrics/expositionformats/DuplicateNamesExpositionTest.java
@@ -13,6 +13,7 @@
 import java.io.IOException;
 import org.junit.jupiter.api.Test;
 
+@SuppressWarnings("deprecation")
 class DuplicateNamesExpositionTest {
 
   private static PrometheusRegistry getPrometheusRegistry() {
diff --git a/prometheus-metrics-instrumentation-caffeine/src/main/java/io/prometheus/metrics/instrumentation/caffeine/CacheMetricsCollector.java b/prometheus-metrics-instrumentation-caffeine/src/main/java/io/prometheus/metrics/instrumentation/caffeine/CacheMetricsCollector.java
@@ -314,6 +314,8 @@ public MetricSnapshots collect() {
   }
 
   @Override
+  @Deprecated
+  @SuppressWarnings("InlineMeSuggester")
   public List<String> getPrometheusNames() {
     if (!collectWeightedSize) {
       return ALL_METRIC_NAMES.stream()
diff --git a/prometheus-metrics-instrumentation-caffeine/src/test/java/io/prometheus/metrics/instrumentation/caffeine/CacheMetricsCollectorTest.java b/prometheus-metrics-instrumentation-caffeine/src/test/java/io/prometheus/metrics/instrumentation/caffeine/CacheMetricsCollectorTest.java
@@ -22,7 +22,7 @@
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.EnumSource;
 
-@SuppressWarnings("CheckReturnValue")
+@SuppressWarnings({"CheckReturnValue", "deprecation"})
 class CacheMetricsCollectorTest {
   // This enum was added to simplify test parametrization on argument options.
   enum Options {
diff --git a/prometheus-metrics-instrumentation-guava/src/main/java/io/prometheus/metrics/instrumentation/guava/CacheMetricsCollector.java b/prometheus-metrics-instrumentation-guava/src/main/java/io/prometheus/metrics/instrumentation/guava/CacheMetricsCollector.java
@@ -217,6 +217,8 @@ public MetricSnapshots collect() {
   }
 
   @Override
+  @Deprecated
+  @SuppressWarnings("InlineMeSuggester")
   public List<String> getPrometheusNames() {
     return ALL_METRIC_NAMES;
   }
diff --git a/prometheus-metrics-instrumentation-guava/src/test/java/io/prometheus/metrics/instrumentation/guava/CacheMetricsCollectorTest.java b/prometheus-metrics-instrumentation-guava/src/test/java/io/prometheus/metrics/instrumentation/guava/CacheMetricsCollectorTest.java
@@ -20,6 +20,7 @@
 import java.util.List;
 import org.junit.jupiter.api.Test;
 
+@SuppressWarnings("deprecation")
 class CacheMetricsCollectorTest {
 
   @Test
diff --git a/prometheus-metrics-model/src/main/java/io/prometheus/metrics/model/registry/Collector.java b/prometheus-metrics-model/src/main/java/io/prometheus/metrics/model/registry/Collector.java
diff --git a/prometheus-metrics-model/src/main/java/io/prometheus/metrics/model/registry/MultiCollector.java b/prometheus-metrics-model/src/main/java/io/prometheus/metrics/model/registry/MultiCollector.java
diff --git a/prometheus-metrics-model/src/main/java/io/prometheus/metrics/model/registry/PrometheusRegistry.java b/prometheus-metrics-model/src/main/java/io/prometheus/metrics/model/registry/PrometheusRegistry.java
diff --git a/prometheus-metrics-model/src/main/java/io/prometheus/metrics/model/snapshots/MetricFamilyDescriptor.java b/prometheus-metrics-model/src/main/java/io/prometheus/metrics/model/snapshots/MetricFamilyDescriptor.java
diff --git a/prometheus-metrics-model/src/test/java/io/prometheus/metrics/model/registry/MultiCollectorNameFilterTest.java b/prometheus-metrics-model/src/test/java/io/prometheus/metrics/model/registry/MultiCollectorNameFilterTest.java
diff --git a/prometheus-metrics-model/src/test/java/io/prometheus/metrics/model/registry/OpenTelemetryExporterRegistryCompatibilityTest.java b/prometheus-metrics-model/src/test/java/io/prometheus/metrics/model/registry/OpenTelemetryExporterRegistryCompatibilityTest.java
diff --git a/prometheus-metrics-model/src/test/java/io/prometheus/metrics/model/registry/PrometheusRegistryTest.java b/prometheus-metrics-model/src/test/java/io/prometheus/metrics/model/registry/PrometheusRegistryTest.java

Original file line number	Diff line number	Diff line change
`@@ -78,6 +78,8 @@ protected MetricSnapshots collectMetricSnapshots(PrometheusScrapeRequest scrapeR`
`78`	`78`	`}`
`79`	`79`
`80`	`80`	`@Override`
	`81`	`+ @Deprecated`
	`82`	`+ @SuppressWarnings("InlineMeSuggester")`
`81`	`83`	`public List<String> getPrometheusNames() {`
`82`	`84`	`List<String> names = new ArrayList<String>();`
`83`	`85`	`names.add("x_calls_total");`
Original file line number	Diff line number	Diff line change
`@@ -90,10 +90,12 @@ protected CounterSnapshot collect(List<Labels> labels, List<DataPoint> metricDat`
`90`	`90`	`for (int i = 0; i < labels.size(); i++) {`
`91`	`91`	`data.add(metricData.get(i).collect(labels.get(i)));`
`92`	`92`	`}`
`93`		`- return new CounterSnapshot(getMetadata(), data);`
	`93`	`+ return new CounterSnapshot(metadata, data);`
`94`	`94`	`}`
`95`	`95`
`96`	`96`	`@Override`
	`97`	`+ @Deprecated`
	`98`	`+ @SuppressWarnings("InlineMeSuggester")`
`97`	`99`	`public MetricType getMetricType() {`
`98`	`100`	`return MetricType.COUNTER;`
`99`	`101`	`}`
Original file line number	Diff line number	Diff line change
`@@ -48,10 +48,12 @@ public CounterSnapshot collect() {`
`48`	`48`	`new CounterSnapshot.CounterDataPointSnapshot(`
`49`	`49`	`value, makeLabels(labelValues), null, 0L));`
`50`	`50`	`});`
`51`		`- return new CounterSnapshot(getMetadata(), dataPoints);`
	`51`	`+ return new CounterSnapshot(metadata, dataPoints);`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`@Override`
	`55`	`+ @Deprecated`
	`56`	`+ @SuppressWarnings("InlineMeSuggester")`
`55`	`57`	`public MetricType getMetricType() {`
`56`	`58`	`return MetricType.COUNTER;`
`57`	`59`	`}`
Original file line number	Diff line number	Diff line change
`@@ -92,10 +92,12 @@ protected GaugeSnapshot collect(List<Labels> labels, List<DataPoint> metricData)`
`92`	`92`	`for (int i = 0; i < labels.size(); i++) {`
`93`	`93`	`dataPointSnapshots.add(metricData.get(i).collect(labels.get(i)));`
`94`	`94`	`}`
`95`		`- return new GaugeSnapshot(getMetadata(), dataPointSnapshots);`
	`95`	`+ return new GaugeSnapshot(metadata, dataPointSnapshots);`
`96`	`96`	`}`
`97`	`97`
`98`	`98`	`@Override`
	`99`	`+ @Deprecated`
	`100`	`+ @SuppressWarnings("InlineMeSuggester")`
`99`	`101`	`public MetricType getMetricType() {`
`100`	`102`	`return MetricType.GAUGE;`
`101`	`103`	`}`
Original file line number	Diff line number	Diff line change
`@@ -52,10 +52,12 @@ public GaugeSnapshot collect() {`
`52`	`52`	`dataPoints.add(`
`53`	`53`	`new GaugeSnapshot.GaugeDataPointSnapshot(value, makeLabels(labelValues), null, 0L));`
`54`	`54`	`});`
`55`		`- return new GaugeSnapshot(getMetadata(), dataPoints);`
	`55`	`+ return new GaugeSnapshot(metadata, dataPoints);`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`@Override`
	`59`	`+ @Deprecated`
	`60`	`+ @SuppressWarnings("InlineMeSuggester")`
`59`	`61`	`public MetricType getMetricType() {`
`60`	`62`	`return MetricType.GAUGE;`
`61`	`63`	`}`
Original file line number	Diff line number	Diff line change
`@@ -647,10 +647,12 @@ protected HistogramSnapshot collect(List<Labels> labels, List<DataPoint> metricD`
`647`	`647`	`for (int i = 0; i < labels.size(); i++) {`
`648`	`648`	`data.add(metricData.get(i).collect(labels.get(i)));`
`649`	`649`	`}`
`650`		`- return new HistogramSnapshot(getMetadata(), data);`
	`650`	`+ return new HistogramSnapshot(metadata, data);`
`651`	`651`	`}`
`652`	`652`
`653`	`653`	`@Override`
	`654`	`+ @Deprecated`
	`655`	`+ @SuppressWarnings("InlineMeSuggester")`
`654`	`656`	`public MetricType getMetricType() {`
`655`	`657`	`return MetricType.HISTOGRAM;`
`656`	`658`	`}`