search

etishor / Metrics.NET

The Metrics.NET library provides a way of instrumenting applications with custom metrics (timers, histograms, counters etc) that can be reported in various ways and can provide insights on what is happening inside a running application.
Apache License 2.0
647 stars 150 forks source link

InfluxDB #80

Open alhardy opened 9 years ago

alhardy commented 9 years ago

I've been playing with the latest InfluxDB, the api has change significantly breaking the current metrics.net influxdb reporter. I'm going to start updating this but before I do how do you foresee managing multiple versions? Do we just have a MetricsReport extensions for each version or maybe separate reporters out to their own assembly?

etishor commented 9 years ago

Since InfluxDB is pre 1.0 version, and api changes are frequent, I would not bother maintaining compatibility with older versions too much. If there is interest in a specific version, or if a version is rather wide-spread then maybe ... but I don't think that is the case with InfluxDB for now.

After 1.0 release it makes sense to have multiple extension method, or maybe a parameter where the user can specify the version.

As long as there are no external dependencies required, I rather keep this reporters in the core library for discover-ability reasons - but I really don't have a strong opinion about this. Maybe initially developed as a separate assembly and merge when dev complete...

Thanks

mariuszwojcik commented 9 years ago

Hi Allan,

Can you let me know what is your progress on influxdb reporter? I am interested in making it work with InfluxDB 0.9. Perhaps we can join forces on that.

timgriffiths commented 9 years ago

+1

mnadel commented 9 years ago

I've opened a PR (https://github.com/etishor/Metrics.NET/pull/110) that implements the new InfluxDB line protocol.

mnadel commented 9 years ago

@etishor Thoughts on PR https://github.com/etishor/Metrics.NET/pull/110? If you're not worried about backwards compatibility until InfluxDB hits 1.0, should be okay to merge? Thanks!

mnadel commented 8 years ago

FYI -- published this to Nuget.org @ https://www.nuget.org/packages/Metrics.NET.InfluxDB/