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

statsd input and field name #602

Closed
lassizci opened this issue Jan 28, 2016 · 2 comments · Fixed by #608
Closed

statsd input and field name #602

lassizci opened this issue Jan 28, 2016 · 2 comments · Fixed by #608
Labels
bug unexpected problem or unintended behavior

Comments

@lassizci
Copy link

I've been trying to template my metrics so that I could have multiple fields with my measurement. According to https://github.com/influxdata/influxdb/tree/master/services/graphite#templates (which is linked from statsd readme page).

it should be possible to make a template like:

templates = [
    "mybycket.* measurement.measurement.field"
]

so this:

echo "mybucket.appstats.foo_count_testing:1|c" | nc -C -w 1 -u localhost 8125

would turn in to:

> select * from mybucket_appstats;
name: mybucket_appstats
------------------------
time            host        metric_type metricgroup foo_count_testing
1453985830000000000 metricstest counter     metrics     1

However, there's only field called "value" and no tag field. So at least with statsd this doesn't seem to work. I'd like to be able to send multiple fields to same measurement.

@tstm
Copy link

tstm commented Jan 28, 2016

+1

@sparrc sparrc added the bug unexpected problem or unintended behavior label Jan 28, 2016
@sparrc
Copy link
Contributor

sparrc commented Jan 28, 2016

thanks for pointing this out 👍, I have a PR #608 up to fix this.

I'm not exactly sure how this would look for timings though, and I'm thinking of simply not supporting fields for timings. This is because (as of 0.10.2) timings will have their actual metrics that are sent in their own fields.

ie, my_timing_metric will be the measurement, with mean, stddev, 90_percentile, etc. as fields

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants