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

Support iterator in Propagator/Extractor/Getter #713

Closed
yurishkuro opened this issue Jul 17, 2020 · 0 comments · Fixed by #825
Closed

Support iterator in Propagator/Extractor/Getter #713

yurishkuro opened this issue Jul 17, 2020 · 0 comments · Fixed by #825
Labels
area:api Cross language API specification issue priority:p2 Medium priority level release:required-for-ga Must be resolved before GA release, or nice to have before GA spec:context Related to the specification/context directory
Projects
GA Spec Burndown

Comments

@yurishkuro
Copy link
Member

Some existing in-band trace formats, like Jaeger baggage, cannot be implemented if the Extractor only allows access to carrier fields via Get(key). The OpenTracing API supported iterator in addition to straight lookup.
@yurishkuro yurishkuro added the spec:context Related to the specification/context directory label Jul 17, 2020
@yurishkuro yurishkuro mentioned this issue Jul 17, 2020
Merged
@carlosalberto carlosalberto added area:api Cross language API specification issue release:required-for-ga Must be resolved before GA release, or nice to have before GA labels Jul 20, 2020
@reyang reyang added the priority:p2 Medium priority level label Jul 24, 2020
@andrewhsu andrewhsu added this to Required for GA, needs action. Add label:release:required-for-ga to issues and PRs when moving them to this column. in GA Spec Burndown Jul 28, 2020
@carlosalberto carlosalberto mentioned this issue Aug 18, 2020
Merged
GA Spec Burndown automation moved this from Required for GA, needs action. Add label:release:required-for-ga to issues and PRs when moving them to this column. to Required for GA, done Aug 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:api Cross language API specification issue priority:p2 Medium priority level release:required-for-ga Must be resolved before GA release, or nice to have before GA spec:context Related to the specification/context directory
Projects
No open projects
GA Spec Burndown
  
Required/Allowed for GA, resolved.
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add Getter's Keys operation. carlosalberto/opentelemetry-specification
3 participants
@carlosalberto @yurishkuro @reyang