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

Drop support for Jaeger Exporter #1711

Closed
Kielek opened this issue Dec 5, 2022 · 3 comments · Fixed by #1896
Closed

Drop support for Jaeger Exporter #1711

Kielek opened this issue Dec 5, 2022 · 3 comments · Fixed by #1896
Assignees
Milestone

Comments

@Kielek
Copy link
Contributor

Kielek commented Dec 5, 2022

As we are in pre 1.0.0 release, it will be good to remove it.
If someone needs it, there is a possibility to load it by plugin.

See open-telemetry/opentelemetry-specification#2859

@Kielek Kielek mentioned this issue Dec 6, 2022
3 tasks
@pellared
Copy link
Member

pellared commented Dec 6, 2022

  1. Jaeger is still very popular
  2. Plugins should be used "as last resort", their primary use case is to allow creating distributions. I think that the users shouldn't not use them unless necessary.
  3. Let's track the issue in the spec. For sure we should make a decision before 1.0.0.

@Kielek Kielek changed the title Drop support for Jaeger Drop support for Jaeger Exporter Dec 7, 2022
@rajkumar-rangaraj rajkumar-rangaraj added this to the 1.0.0-rc milestone Dec 7, 2022
@RassK
Copy link
Contributor

RassK commented Dec 7, 2022

Doesn't seem as a fat dependency in SDK. Since we also load it if we need it, there is not many issues keeping it until deprecation notice is official.

Plugins should be used "as last resort", their primary use case is to allow creating distributions. I think that the users shouldn't not use them unless necessary.

Especially true without integration layer. Having integration through magic methods could cause loads of issues. At least distributors know more what's going on.

@pellared
Copy link
Member

Deprecation note is official open-telemetry/opentelemetry-specification#2858

I think we are good to remove it. Removing the "milestone" to discuss if we want to make it sooner.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

5 participants