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

[v2] Rename "metric" entity to "metricset" #1356

Closed
axw opened this issue Sep 5, 2018 · 5 comments
Closed

[v2] Rename "metric" entity to "metricset" #1356

axw opened this issue Sep 5, 2018 · 5 comments

Comments

@axw
Copy link
Member

axw commented Sep 5, 2018

The metrics API defines a metric entity, which is really a set of metrics. I propose that we rename this to "metricset", or something similar. Calling it "metric" might give the impression that it is a singular measurement.

@watson
Copy link
Contributor

watson commented Sep 5, 2018

How about just metrics (plural). I try to stick to one-word names if I can in these cases. But maybe I misunderstood the use-case here...

@axw
Copy link
Member Author

axw commented Sep 6, 2018

This is for v2 objects, e.g.

{"metadata":{}}
{"transaction":{}}
{"metrics":{"tags":{}, "samples":{}}}

"metrics" as an object name seems a bit odd to me, but better than just "metric". We could flatten the structure so we fold "tags" into each entry of "samples", but that would lead to a lot of repetition.

@roncohen
Copy link
Contributor

roncohen commented Sep 6, 2018

I like metricset

@watson
Copy link
Contributor

watson commented Sep 6, 2018

I'm fine either way. Just wanted to throw in the other option 😃

@roncohen roncohen mentioned this issue Sep 6, 2018
30 tasks
@roncohen
Copy link
Contributor

roncohen commented Sep 6, 2018

metricbeat uses 'metricset' as well by the way: https://www.elastic.co/guide/en/beats/metricbeat/current/metricbeat-modules.html

@roncohen roncohen changed the title Rename "metric" entity to "metricset" [v2] Rename "metric" entity to "metricset" Sep 6, 2018
roncohen pushed a commit to roncohen/apm-server that referenced this issue Sep 6, 2018
roncohen added a commit that referenced this issue Sep 10, 2018
Rename json schemas as well

Closes #1356.
roncohen added a commit to roncohen/apm-server that referenced this issue Oct 7, 2018
roncohen added a commit to roncohen/apm-server that referenced this issue Oct 15, 2018
roncohen added a commit to roncohen/apm-server that referenced this issue Oct 15, 2018
roncohen added a commit to roncohen/apm-server that referenced this issue Oct 15, 2018
roncohen added a commit to roncohen/apm-server that referenced this issue Oct 15, 2018
roncohen added a commit to roncohen/apm-server that referenced this issue Oct 15, 2018
roncohen added a commit to roncohen/apm-server that referenced this issue Oct 16, 2018
roncohen added a commit that referenced this issue Oct 16, 2018
Rename json schemas as well

Closes #1356.
bmorelli25 pushed a commit to bmorelli25/observability-docs that referenced this issue Dec 18, 2023
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

4 participants