Influxclient broken due to changes in influxdb repo about go-metrics HOT 4 CLOSED

This is due to substantial changes in influxdb (master) client, it's api and protocol (urls, params etc.) it speaks with a server. The changes are incompatible with any released influxdb server AFAIK.
I have a patch in my fork, this is the diff: https://github.com/rzab/go-metrics/compare/rcrowley:master...master I'm not sure if I should PR this: the new influxdb api is anything but stable.

from go-metrics.

I'll reiterate a comment I made a while ago - I think all the output clients should have their own repos, and this repo should merely aim to be vigorous in maintaining a reference of existing extensions. This applies to my own librato client as well, I just have no had the cycles to rip it out. This seems like a good opportunity to rip out the influx one, especially if the influxdb API itself is still not stable.

from go-metrics.

@mihasya I must say, that I agree with you. While the send function in https://github.com/rcrowley/go-metrics/blob/master/influxdb/influxdb.go may still exist as a data wrapper, the client itself should not be part of this implementation.

from go-metrics.

Closing this as per-sink clients in this repo are not a sustainable way to move forward. I encourage you to create a separate client repo which imports this one and adds a reporter that works with whichever version of the influx API you require. Sorry it took so long to actually process this.

from go-metrics.