Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Disable upstream default exporter (for now) #894

Merged
merged 2 commits into from
Aug 6, 2024

Conversation

breedx-splk
Copy link
Contributor

@breedx-splk breedx-splk commented Aug 6, 2024

Resolves #893

Upstream provides the LoggingSpanExporter as the default. For some configurations, this can cause a problem, so we disable it here. Our custom BatchSpanProcessor will take care of passing things to our exporter.

@breedx-splk breedx-splk requested review from a team as code owners August 6, 2024 04:16
@breedx-splk breedx-splk merged commit ee9a58e into signalfx:main Aug 6, 2024
3 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Aug 6, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LoggingSpanExporter should be opt-in
2 participants