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

Remove or rewrite api-tracing section "Runtimes with multiple deployments/applications" #272

Closed
yurishkuro opened this issue Sep 29, 2019 · 2 comments
Closed

Remove or rewrite api-tracing section "Runtimes with multiple deployments/applications" #272

yurishkuro opened this issue Sep 29, 2019 · 2 comments
Labels
release:after-ga Not required before GA release, and not going to work on before GA spec:trace Related to the specification/trace directory
Milestone
v0.6

Comments

@yurishkuro
Copy link
Member

Runtimes that support multiple deployments or applications might need to provide a different Tracer instance to each deployment.

Not clear what "multiple deployments" means here, could give some examples.

The following references to Provider and ServiceLoader seem out of place completely, unrelated to the API, which is now TracerFactory per OTEP 16.
@yurishkuro yurishkuro mentioned this issue Sep 29, 2019
Merged
@bogdandrutu bogdandrutu added this to the Alpha v0.2 milestone Sep 30, 2019
@carlosalberto carlosalberto modified the milestones: Alpha v0.3, Alpha v0.4 Jan 28, 2020
@carlosalberto carlosalberto modified the milestones: v0.5, v0.6 Jun 9, 2020
@bogdandrutu bogdandrutu added the spec:trace Related to the specification/trace directory label Jun 12, 2020
@carlosalberto
Copy link
Contributor

I'm a little bit confused here (we should have reviewed this issue long time ago ;) )

@yurishkuro could you comment if this is still an issue, and if it is, could you elaborate?

@carlosalberto carlosalberto added the release:after-ga Not required before GA release, and not going to work on before GA label Jul 2, 2020
@yurishkuro
Copy link
Member Author

This exact wording has been removed, can close this.

TuckTuckFloof pushed a commit to TuckTuckFloof/opentelemetry-specification that referenced this issue Oct 15, 2020
@HudsonHumphries
Use Struct as Key in Processor (open-telemetry#272)
47d5a55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
release:after-ga Not required before GA release, and not going to work on before GA spec:trace Related to the specification/trace directory
Projects
None yet
Milestone
v0.6
Development

No branches or pull requests
4 participants
@carlosalberto @bogdandrutu @yurishkuro @SergeyKanzhelev