-
Notifications
You must be signed in to change notification settings - Fork 807
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
port stackdriver trace exporter from OpenCensus. #530
Comments
Not yet, but we can port stackdriver trace exporter from OpenCensus. |
I'll take this |
The reference implementation has both |
Depending on how much code is shared I would prefer to actually keep them as separate packages. If a customer only needs one of them we are unnecessarily increasing their deployment size if they are together. If there is shared code, we could make it a separate package?
All three could live in |
AFAIK only resources(common-utils) stuff is shared between these two.
I completely agree with this and looks like OpenTelemetry Go also implemented like that : https://github.com/open-telemetry/opentelemetry-go/tree/master/exporter. |
Closing via #648 |
Does one exist for js?
The text was updated successfully, but these errors were encountered: