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.

Sign up for GitHub

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

Jump to bottom

Summary is lacking all details in the DataModel specificaiton #2000

Closed
jsuereth opened this issue Oct 7, 2021 · 0 comments · Fixed by #2002
Closed

Summary is lacking all details in the DataModel specificaiton #2000

jsuereth opened this issue Oct 7, 2021 · 0 comments · Fixed by #2002
Assignees
@jsuereth @SergeyKanzhelev
Labels
area:data-model For issues related to data model spec:metrics Related to the specification/metrics directory

Comments

@jsuereth
Copy link
Contributor

jsuereth commented Oct 7, 2021

We deferred addressing summary as a legacy type, relying on protocol buffer documentation.

We need to address some specifics so that implementations using summary can accurately fill out fields in opentelemetry, e.g. open-telemetry/opentelemetry-collector-contrib#5667

cc @jmacd
@jsuereth jsuereth added spec:metrics Related to the specification/metrics directory area:data-model For issues related to data model labels Oct 7, 2021
@jsuereth jsuereth self-assigned this Oct 7, 2021
@jsuereth jsuereth mentioned this issue Oct 7, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jsuereth jsuereth

@SergeyKanzhelev SergeyKanzhelev

Labels
area:data-model For issues related to data model spec:metrics Related to the specification/metrics directory
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fill out Summary metric details jsuereth/opentelemetry-specification
2 participants
@jsuereth @SergeyKanzhelev