-
Notifications
You must be signed in to change notification settings - Fork 863
Closed as not planned
Labels
StaleIssues and pull requests which have been flagged for closing due to inactivityIssues and pull requests which have been flagged for closing due to inactivityenhancementNew feature or requestNew feature or requestpkg:OpenTelemetryIssues related to OpenTelemetry NuGet packageIssues related to OpenTelemetry NuGet package
Description
Package
OpenTelemetry
Is your feature request related to a problem?
BatchExportProcessor is unnecessarily spawning a thread for the export. This thread spends 99% of its time blocked. This is a little waste of resource and it can also mislead investigations about thread pool starvation (a blocked thread looks like an issue until you find out it's not a thread pool thread).
What is the expected behavior?
The export spends its time waiting for a trigger or for the response of the export operation. It is completely fine using the .NET thread pool as long as all operations are asynchronous. This would make OTEL a better citizen in the .NET ecosystem.
The work would be:
- remove the exporter thread
- migrate BaseExporter.Export to async. Or probably add a new method to avoid a breaking change
Which alternative solutions or features have you considered?
N/A
Additional context
No response
cijothomas and KrzysztofBranicki
Metadata
Metadata
Assignees
Labels
StaleIssues and pull requests which have been flagged for closing due to inactivityIssues and pull requests which have been flagged for closing due to inactivityenhancementNew feature or requestNew feature or requestpkg:OpenTelemetryIssues related to OpenTelemetry NuGet packageIssues related to OpenTelemetry NuGet package