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.

Sign up for GitHub

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

Jump to bottom

Refactor duplicate code in the OTLP exporter #2015

Closed
MrAlias opened this issue Jun 17, 2021 · 3 comments
Closed

Refactor duplicate code in the OTLP exporter #2015

MrAlias opened this issue Jun 17, 2021 · 3 comments
Labels
help wanted Extra attention is needed pkg:exporter:otlp Related to the OTLP exporter package

Comments

@MrAlias
Copy link
Contributor

MrAlias commented Jun 17, 2021

In #1827 the OTLP exporter was split by signal. The split resulted in large amounts of unexported code being duplicated (e.g. transform, connection, otlpconfig). Refactor and unification this code is needed.
@MrAlias MrAlias added help wanted Extra attention is needed release:after-ga pkg:exporter:otlp Related to the OTLP exporter package labels Jun 17, 2021
@srikanthccv
Copy link
Member

@MrAlias I would like to pick this up.

@MadVikingGod
Copy link
Contributor

Looks like #2024 covers part of this. I doubt it is everything.

@srikanthccv
Copy link
Member

Looks like #2024 covers part of this. I doubt it is everything.

Looks like it has overlap work. I think another common piece of functionality is config.

@MrAlias MrAlias mentioned this issue Aug 6, 2021
Merged
@MrAlias MrAlias mentioned this issue Dec 3, 2021
Merged
@MrAlias MrAlias mentioned this issue Dec 3, 2021
Merged
@MrAlias MrAlias closed this as completed Sep 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Extra attention is needed pkg:exporter:otlp Related to the OTLP exporter package
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@MadVikingGod @MrAlias @srikanthccv