Fixed statsd histogram implementation (#38)

avast · Nov 7, 2018 · 2f42109 · 2f42109
1 parent dd5ca01
commit 2f42109
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 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
@@ -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
@@ -18,6 +18,6 @@ public void testSendsValues() {
             histogram.update(10);
         }
 
-        verify(client, times(5)).recordSetValue(name, "10");
+        verify(client, times(5)).histogram(name, 10);
     }
 }