haskell-github-trust / ekg-statsd

Flush system metrics to statsd
BSD 3-Clause "New" or "Revised" License
26 stars 23 forks source link

Report counters in an incremental fashion. #23

Closed j6carey closed 4 years ago

j6carey commented 5 years ago

Commit e444d8b680e92c4e8a45319ee41956433025b28d eliminated the suppression of messages that were in some sense redundant, but as a side effect it started reporting the current counter value instead of the increment since the last report.

This change causes only the increment to be reported--which is what statsd servers expect.

Gauges are unaffected. This change affects only counters, but that includes the counter within each distribution.

23Skidoo commented 5 years ago

/cc @adinapoli-iohk

23Skidoo commented 5 years ago

@adinapoli-iohk Ping.

j6carey commented 4 years ago

@23Skidoo , @adinapoli-iohk , I merged master.

23Skidoo commented 4 years ago

@j6carey Can you please add a notice to the changelog? nvm, did that myself

23Skidoo commented 4 years ago

Merged, thanks!

23Skidoo commented 4 years ago

Released on Hackage as ekg-statsd-0.2.5.0.

j6carey commented 4 years ago

@23Skidoo , thanks!