{"action":"closed","issue":{"url":"https://api.github.com/repos/indeedeng/java-dogstatsd-client/issues/16","labels_url":"https://api.github.com/repos/indeedeng/java-dogstatsd-client/issues/16/labels{/name}","comments_url":"https://api.github.com/repos/indeedeng/java-dogstatsd-client/issues/16/comments","events_url":"https://api.github.com/repos/indeedeng/java-dogstatsd-client/issues/16/events","html_url":"https://github.com/indeedeng/java-dogstatsd-client/issues/16","id":99512571,"number":16,"title":"recordExecutionTime doesn't convert from milliseconds to seconds","user":{"login":"jlucier-GiantMedia","id":12616564,"avatar_url":"https://avatars.githubusercontent.com/u/12616564?v=3","gravatar_id":"","url":"https://api.github.com/users/jlucier-GiantMedia","html_url":"https://github.com/jlucier-GiantMedia","followers_url":"https://api.github.com/users/jlucier-GiantMedia/followers","following_url":"https://api.github.com/users/jlucier-GiantMedia/following{/other_user}","gists_url":"https://api.github.com/users/jlucier-GiantMedia/gists{/gist_id}","starred_url":"https://api.github.com/users/jlucier-GiantMedia/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jlucier-GiantMedia/subscriptions","organizations_url":"https://api.github.com/users/jlucier-GiantMedia/orgs","repos_url":"https://api.github.com/users/jlucier-GiantMedia/repos","events_url":"https://api.github.com/users/jlucier-GiantMedia/events{/privacy}","received_events_url":"https://api.github.com/users/jlucier-GiantMedia/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-08-06T20:00:11Z","updated_at":"2015-08-09T12:46:54Z","closed_at":"2015-08-09T12:46:54Z","body":"I'm using this library and I need to report runtimes to datadog. The readme claims that passing in ```long timeInMs``` will get converted to fractional seconds. From using the library and reading the source, it looks like this is not the case (this is however the case in the tim-group repo https://github.com/tim-group/java-statsd-client). If this could be fixed that would be ideal, but for the time being the specification in the readme should be updated. Thank You.\r\n\r\nMy use case:\r\n```java\r\nInstant start = Instant.now();\r\n... operations ...\r\nMain.statsD.time(\"runtime\", Duration.between(start, Instant.now()).toMillis());\r\n```\r\n\r\nMy datadog graph shows runtime in the range of 13 seconds, but this operation is used in a web environment and server response is ~25 ms. Thus, 13 ms makes much more sense."}}