Code review comment for lp:~ricardokirkner/txstatsd/gauge-with-delta

Revision history for this message
Ricardo Kirkner (ricardokirkner) wrote :

> > this api seems like counters, have you looked at that?
>
> Haven't looked into txstatsd counters in detail yet, but will do. I added this
> gauge delta support because it's supported by the "official" statsd server, so
> clients (other than txstatsd) are starting to support it.

counters are supposed to be reset to 0 every 10s (ie, after every flush)... gauges are not; if the value didn't change statsd will send the latest available value.
Even if counters implement the same api as gauges for this in txstatsd, it's not possible to use non-txstatsd clients with a txstatsd server as the standard api for counters in statsd doesn't support deltas. This is why I'm adding delta support for gauges (so that we can use other statsd clients with a txstatsd server)

« Back to merge proposal