From 2f4210951e8864439468475587a04bb636e7a529 Mon Sep 17 00:00:00 2001 From: Honza Strnad Date: Wed, 7 Nov 2018 08:53:59 +0100 Subject: [PATCH] Fixed statsd histogram implementation (#38) --- .../main/java/com/avast/metrics/statsd/StatsDHistogram.java | 3 +-- .../java/com/avast/metrics/statsd/StatsDHistogramTest.java | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/statsd/src/main/java/com/avast/metrics/statsd/StatsDHistogram.java b/statsd/src/main/java/com/avast/metrics/statsd/StatsDHistogram.java index 32846c9..dbf054a 100755 --- a/statsd/src/main/java/com/avast/metrics/statsd/StatsDHistogram.java +++ b/statsd/src/main/java/com/avast/metrics/statsd/StatsDHistogram.java @@ -14,8 +14,7 @@ public StatsDHistogram(final StatsDClient client, final String name) { @Override public void update(final long value) { - // TODO: client.histogram() supports sample rate, what is recordSetValue? - client.recordSetValue(name, String.valueOf(value)); + client.histogram(name, value); } @Override diff --git a/statsd/src/test/java/com/avast/metrics/statsd/StatsDHistogramTest.java b/statsd/src/test/java/com/avast/metrics/statsd/StatsDHistogramTest.java index 89bd384..299d96d 100644 --- a/statsd/src/test/java/com/avast/metrics/statsd/StatsDHistogramTest.java +++ b/statsd/src/test/java/com/avast/metrics/statsd/StatsDHistogramTest.java @@ -18,6 +18,6 @@ public void testSendsValues() { histogram.update(10); } - verify(client, times(5)).recordSetValue(name, "10"); + verify(client, times(5)).histogram(name, 10); } }