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

Rename opentelemetry-ext-otcollector to opentelemetry-ext-opencensus #541

Closed
owais opened this issue Apr 2, 2020 · 1 comment · Fixed by #695
Closed

Rename opentelemetry-ext-otcollector to opentelemetry-ext-opencensus #541

owais opened this issue Apr 2, 2020 · 1 comment · Fixed by #695
Assignees

Comments

@owais
Copy link
Contributor

owais commented Apr 2, 2020

I was trying some example apps locally and was surprised that otcollector exporter uses opencensus internally instead of otlp. I realize now that the plan was to implement opencensus temporarily and eventually migrate to opentelemetry. That makes sense but I think this package should just be renamed to opentelemetry-ext-opencensus as it actually implements an exporter that speaks the OpenCensus protocol. It is not specifically built to export only to OT collector.

Even if we completely drop support for the OpenCensus protocol when we add support for OTLP, I think it should not happen in the same package but in a new package called opentelemetry-ext-otlp instead. This is because OTLP is an open protocol that anyone can implement. opentelemetry-python can export to any service that accepts OTLP, not just the OpenTelemetry collector.

@nirsky
Copy link
Contributor

nirsky commented May 11, 2020

Is there any plan for implementing the otlp exporter?

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

Successfully merging a pull request may close this issue.

4 participants