-
Notifications
You must be signed in to change notification settings - Fork 495
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Contact client implementation authors about splitting off into new libraries #121
Comments
From the gist of it, it sounds like a good idea, maybe you could share some details you have in mind? |
Howdy! Basically, just spinning the individual reporters that require an external dependency (like a service) into their own little repos that import |
@mihasya Any preferences on the naming of theses "spin-off" ? |
@cyberdelia hmm.. not entirely sure, but I also don't think it's worth agonizing over. They'll theoretically only be imported once, in |
@cyberdelia probably something like |
@cyberdelia you can see my stab at the Librato client's move here #122 |
For graphite: https://github.com/cyberdelia/go-metrics-graphite |
@cyberdelia nice and easy. Thanks! |
I don't use the stathat lib at all, if someone else wants to take ownership - I'm happy to relinquish any "control" |
@bketelsen thanks for letting us know. We'll find it a home, even if it's just under my account with a "MAINTAINER WANTED" in the README. |
It looks like the github account for Michael Crosby is @crosbymichael. Michael, would you be interested in becoming the maintainer for a split-out version of the influx reporter? |
@mihasya ya, i can take the influx part |
@crosbymichael Beauty! Please check out the repos myself and @cyberdelia have created for librato and graphite |
http://prometheus.io/ looks like a candidate for the creation of a client |
Is there something inherently incompatible with statsd and this library, that there isn't an exporter for it? |
Other libraries to document:
The text was updated successfully, but these errors were encountered: