search

rcrowley / go-metrics

Go port of Coda Hale's Metrics library
Other
3.45k stars 494 forks source link

Bosun Support (aka OpenTSDB api) #131

Open rbucker opened 9 years ago

rbucker commented 9 years ago

The current OpenTSDB seems to implement the socket version of messaging. It's very similar to the Graphite version. Looking at the Bosun documentation (said to support OpenTSDB 2.x) it appears to be limited to the http POST verb. I'm about to fork go-metrics in order to implement the feature but I wanted to get your opinion first.

Is Bosun supported? Is someone working on it? Would you accept a pull request?

mihasya commented 9 years ago

@rbucker you come to this repo at an exciting time! Additional clients will not be accepted to the core go-metrics repository, as it creates a support nightmare as external APIs change and the maintainers have to try and make decisions about clients they don't use which potentially break things for people in communities to which the maintainers don't belong. As such, the decision was made to encourage "output" clients to be implemented as standalone repos. You can see the early examples here and here and the related issue at #121.

I'll gladly accept a pull request that updates the README to include bosun as a supported client with a link to the corresponding repository.

rbucker commented 9 years ago

It's a crappy answer but I guess I'll have to live with it. I've created the wrapper for the project. I should have a working model tonight.

mihasya commented 9 years ago

Sorry you find the solution "crappy." I look forward to adding a link to your repo when it is ready.

rbucker commented 9 years ago

Code Complete here

please update whatever doc needs it.... and close.