-
Notifications
You must be signed in to change notification settings - Fork 827
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
Simplify OpenTracingShim creational patterns #3084
Comments
I might rename this to just "simplify OpenTracingShim creational patterns" because we might want to go with a builder. |
Hi, can I take this issue? cc @alolita |
Thanks @IrisTuntun for your interest in this issue! I think we should hold off temporarily until getting some input on this spec issue I filed open-telemetry/opentelemetry-specification#1608 I am hoping we can reduce to just a single factory |
Resolved in #5110. |
Introduce builder or
instead of 4 leave only two by removing
createTracerShim(OpenTelemetry openTelemetry)
andcreateTracerShim()
fromOpenTracingShim
classThe text was updated successfully, but these errors were encountered: