search

github / brubeck

A Statsd-compatible metrics aggregator
MIT License
1.19k stars 94 forks source link

Not compliant with statsd spec wrt counters and meters #50

Open deanbittner opened 8 years ago

deanbittner commented 8 years ago

The spec here -> https://github.com/b/statsd_spec <- indicates that counters are indicated by 'c' and meters by 'm'.

The statsd sampler indicates counters as 'C', meters as 'c', and fails to recognize 'm'.

deanbittner commented 8 years ago

Not sure what 'C' is in brubeck but it corresponds to what internally is called a counter. The 'c' is internally a meter, and 'm' is not present. I believe it's fairly benign to treat 'c' as a meter and add 'm' to be treated the same way.