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

Support OpenCensus -> OpenTelemetry migration path #2012

Closed
nilebox opened this issue Nov 5, 2020 · 1 comment
Closed

Support OpenCensus -> OpenTelemetry migration path #2012

nilebox opened this issue Nov 5, 2020 · 1 comment
Labels
Feature Request Suggest an idea for this project

Comments

@nilebox
Copy link
Member

nilebox commented Nov 5, 2020

Is your feature request related to a problem? Please describe.
The OpenTelemetry merger announcement promised straightforward backwards compatibility with OpenCensus.

We need to provide support for libraries which are currently instrumented via OpenCensus APIs to deliver their data to OpenTelemetry pipeline.

Describe the solution you'd like
#1863 Is the first "shim" version which has been merged.
It has a problem of using expiring cache which may not work well for use cases with high QPS.
We may try to implement TracerImpl which wouldn't use any "impl" code from OpenCensus at all (or use it minimally without requiring a cache).

Additional context
See discussion in #1863 for more information.

@nilebox nilebox added the Feature Request Suggest an idea for this project label Nov 5, 2020
@bogdandrutu
Copy link
Member

@nilebox can this be closed as duplicate of #2013 or the other one, and move the comments to here?

@nilebox nilebox closed this as completed Nov 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request Suggest an idea for this project
Projects
None yet
Development

No branches or pull requests

2 participants