Merge pull request #8 from finn-no/bugfix/vanilla-statsd-time-semantics

restores "recordExecutionTime" compatibility with vanilla statsd

Author: camerondavison

.gitignore

@@ -1,3 +1,4 @@
 bin
 build
-*.iml
+*.iml
+/target/

src/main/java/com/timgroup/statsd/NonBlockingStatsDClient.java

@@ -374,7 +374,7 @@ public void gauge(String aspect, long value, String... tags) {
      */
     @Override
     public void recordExecutionTime(String aspect, long timeInMs, String... tags) {
-        recordHistogramValue(aspect, (timeInMs * 0.001), tags);
+        send(String.format("%s%s:%d|ms%s", prefix, aspect, timeInMs, tagString(tags)));
     }
 
     /**

src/test/java/com/timgroup/statsd/NonBlockingStatsDClientTest.java

@@ -214,7 +214,7 @@ public void stop() throws Exception {
         client.recordExecutionTime("mytime", 123);
         server.waitForMessage();
 
-        assertThat(server.messagesReceived(), contains("my.prefix.mytime:0.123|h"));
+        assertThat(server.messagesReceived(), contains("my.prefix.mytime:123|ms"));
     }
 
     /**
@@ -235,7 +235,7 @@ public void stop() throws Exception {
             client.recordExecutionTime("mytime", 123, "foo:bar", "baz");
             server.waitForMessage();
 
-            assertThat(server.messagesReceived(), contains("my.prefix.mytime:0.123|h|#baz,foo:bar"));
+            assertThat(server.messagesReceived(), contains("my.prefix.mytime:123|ms|#baz,foo:bar"));
         } finally {
             // reset the default Locale in case changing it has side-effects
             Locale.setDefault(originalDefaultLocale);
@@ -250,7 +250,7 @@ public void stop() throws Exception {
         client.recordExecutionTime("mytime", 123, "foo:bar", "baz");
         server.waitForMessage();
 
-        assertThat(server.messagesReceived(), contains("my.prefix.mytime:0.123|h|#baz,foo:bar"));
+        assertThat(server.messagesReceived(), contains("my.prefix.mytime:123|ms|#baz,foo:bar"));
     }