-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Add NewExportPipeline and InstallNewPipeline functions to OTLP exporter #1347
Comments
Hello, i would to help with this issue, looks like open-telemetry/opentelemetry-collector#2159 can be taken as reasonable defaults. |
@binjip978 sorry about the late response, thanks for picking this issue up. I think there might be some differences between what the other exporters do and what that example does. We should try to remain consistent with what the other examples do. |
👋 looking through the linked code and wanting to understand the intention of this change since all the examples are slightly different. @MrAlias could you possibly summarize the behavior of these methods for |
This issue is outdated at this point. Closing. |
Similar to the other exporters:
opentelemetry-go/exporters/metric/prometheus/prometheus.go
Line 90 in 3d6e8b5
opentelemetry-go/exporters/stdout/exporter.go
Line 53 in fd3c82b
opentelemetry-go/exporters/trace/jaeger/jaeger.go
Line 154 in fd3c82b
opentelemetry-go/exporters/trace/zipkin/zipkin.go
Line 114 in fd3c82b
add the NewExportPipeline and InstallNewPipeline convince functions.
The text was updated successfully, but these errors were encountered: