open-telemetry / opentelemetry-collector-contrib

Contrib repository for the OpenTelemetry Collector
https://opentelemetry.io
Apache License 2.0
3.02k stars 2.34k forks source link

[exporter/elasticsearch] Use a single instance of `esExporter` for all event types #33326

Closed lahsivjar closed 3 weeks ago

lahsivjar commented 4 months ago

Component(s)

exporter/elasticsearch

Describe the issue you're reporting

With https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/33318 both logs and traces exporter will be unified to use the newly introduced esExporter, however, it will still create a new instance of the esExporter for each event type. This leads to additional overhead to maintain more bulk indexers. Refactor the code using SharedComponent or a similar method to create a single instance of the esExporter for all event types.

github-actions[bot] commented 4 months ago

Pinging code owners:

github-actions[bot] commented 2 months ago

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

github-actions[bot] commented 3 weeks ago

This issue has been closed as inactive because it has been stale for 120 days with no activity.