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

Remove exporterqueue.Factory and simplify exportehelper.WithQueueRequest #12509

Merged

bogdandrutu merged 1 commit into open-telemetry:main from bogdandrutu:new-queue-factory

Feb 28, 2025

Merged

Remove exporterqueue.Factory and simplify exportehelper.WithQueueRequest #12509

Cleanup Queue factory and initialization

Codecov / codecov/patch failed Feb 28, 2025 in 0s

87.17% of diff hit (target 95.00%)

View this Pull Request on Codecov

87.17% of diff hit (target 95.00%)

Annotations

Check warning on line 54 in exporter/exporterhelper/common.go

codecov / codecov/patch

exporter/exporterhelper/common.go#L53-L54

Added lines #L53 - L54 were not covered by tests

Check warning on line 48 in exporter/exporterhelper/logs.go

codecov / codecov/patch

exporter/exporterhelper/logs.go#L48

Added line #L48 was not covered by tests

Check warning on line 47 in exporter/exporterhelper/metrics.go

codecov / codecov/patch

exporter/exporterhelper/metrics.go#L47

Added line #L47 was not covered by tests

Check warning on line 47 in exporter/exporterhelper/traces.go

codecov / codecov/patch

exporter/exporterhelper/traces.go#L47

Added line #L47 was not covered by tests

Check warning on line 50 in exporter/exporterhelper/xexporterhelper/profiles.go

codecov / codecov/patch

exporter/exporterhelper/xexporterhelper/profiles.go#L50

Added line #L50 was not covered by tests

View more details on Codecov