search

davidB / metrics-influxdb

A reporter for metrics which announces measurements to an InfluxDB server.
Other
270 stars 102 forks source link

Allow not sending some parts of the metrics #36

Open shanielh opened 8 years ago

shanielh commented 8 years ago

Hi, I would like to send only some parts of the metrics.

for example, for timers I don't want to send the one_minute, five_minute and fifteen_minute mean_rate and run_count. Those values does not matter to me because I'm using HdrHistogramResetOnSnapshotReservoir (see https://bitbucket.org/marshallpierce/hdrhistogram-metrics-reservoir and http://taint.org/2014/01/16/145944a.html) which resets everything every time a snapshot is taken, So the sample size is the rate since last report.

If you'd like I can do a pull request by my own but please guide me where do you want me to make the change.

Thanks.

davidB commented 8 years ago

PR are welcome. I have to go back into the code before to provide some guide. I'll take a look asap.

On Thu, Feb 25, 2016 at 8:12 AM, Shani Elharrar notifications@github.com wrote:

Hi, I would like to send only some parts of the metrics.

for example, for timers I don't want to send the one_minute, five_minute and fifteen_minute mean_rate and run_count. Those values does not matter to me because I'm using HdrHistogramResetOnSnapshotReservoir (see https://bitbucket.org/marshallpierce/hdrhistogram-metrics-reservoir and http://taint.org/2014/01/16/145944a.html) which resets everything every time a snapshot is taken, So the sample size is the rate since last report.

If you'd like I can do a pull request by my own but please guide me where do you want me to make the change.

Thanks.

— Reply to this email directly or view it on GitHub https://github.com/davidB/metrics-influxdb/issues/36.

shanielh commented 8 years ago

@davidB Looking forward for your reply :+1: