Skip to content
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

Tag support #21

Open
esword opened this issue Jul 23, 2015 · 5 comments
Open

Tag support #21

esword opened this issue Jul 23, 2015 · 5 comments

Comments

@esword
Copy link

esword commented Jul 23, 2015

I have seen a couple different ways that statsd-to-X bridges enabled support for tagging. This project does it by looking for tags embedded as part of the metric name. It is ugly, but probably safest since it makes no assumption about how the statsD process looks at data after the type constant.

A cleaner way is how DataDog does it (see format). I have used this
fork of a popular statsd client library to push data in this way, but it may assume that the DD agent is the statsD service (rather than a generic statsD service that would barf on the data after the type data).

Just tossing ideas out. We are looking at using influxDB and need a statsD-with-tags solution. I can't offer to help (yet), but maybe someone else can get a jump on things using either of these approaches as a basis.

@esword
Copy link
Author

esword commented Jul 23, 2015

Another possibility: Here's a thread from the dropwizard metrics forum about what that project may do for tag support - (https://groups.google.com/forum/#!topic/metrics-user/95Dv1xcOox0)

@aderumier
Copy link

Hi,
I need this feature too, I'll try to implement something similar to https://github.com/danslimmon/statsd-opentsdb-backend.

also, influxdb 0.9 need to use the new line protocol to pass tags (no more json), https://influxdb.com/docs/v0.9/write_protocols/write_syntax.html

So, it's not too much different than opentsb

@aderumier
Copy link

Seem that somebody already done it in this fork :)

Asana@cc75147

@aderumier
Copy link

here another implementation : starsy@a0c97e5

@rjaquino
Copy link

rjaquino commented Sep 8, 2015

The second implementation has also been issued as a PR here: #22. I had meant to submit ours as a PR once we confirmed it was working for us, but the line protocol seems strictly cleaner.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants