search

tim-group / java-statsd-client

a java statsd client library
Other
275 stars 139 forks source link

use documented packet format when sampling is enabled #29

Closed znull closed 9 years ago

znull commented 9 years ago

The sample rate is documented to be delimited by "|@", not "@" as was previously used.

https://github.com/etsy/statsd/blob/master/docs/metric_types.md#sampling

scarytom commented 9 years ago

This is a good spot. Do you happen to know when this changed?

znull commented 9 years ago

I don't, I just know it wasn't working and this fixes it. (Before this, instead of getting the usual set of statsd metrics, I'd just get "count" and "rate").

znull commented 9 years ago

@scarytom Would it be possible to push this to maven central as part of a 3.0.3 release? That would make my life easier.

scarytom commented 9 years ago

I've just released v3.1.0 with these changes.