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

Add NewExportPipeline and InstallNewPipeline functions to OTLP exporter #1347

Closed
MrAlias opened this issue Nov 18, 2020 · 4 comments
Closed
Labels
help wanted Extra attention is needed pkg:exporter:otlp Related to the OTLP exporter package

Comments

@MrAlias
Copy link
Contributor

MrAlias commented Nov 18, 2020

Similar to the other exporters:

add the NewExportPipeline and InstallNewPipeline convince functions.

@binjip978
Copy link
Contributor

Hello, i would to help with this issue, looks like open-telemetry/opentelemetry-collector#2159 can be taken as reasonable defaults.

@MrAlias
Copy link
Contributor Author

MrAlias commented Dec 1, 2020

@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.

@MrAlias MrAlias added pkg:exporter:otlp Related to the OTLP exporter package and removed pkg:exporter labels Apr 6, 2021
@MrAlias MrAlias added the help wanted Extra attention is needed label May 3, 2022
@eggyhead
Copy link

eggyhead commented Oct 7, 2022

👋 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 oltp?

@MrAlias
Copy link
Contributor Author

MrAlias commented Oct 12, 2022

This issue is outdated at this point. Closing.

@MrAlias MrAlias closed this as completed Oct 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed pkg:exporter:otlp Related to the OTLP exporter package
Projects
None yet
Development

No branches or pull requests

3 participants