Provide string representation for Span Status field

open-telemetry / opentelemetry-collector-contrib

Contrib repository for the OpenTelemetry Collector

https://opentelemetry.io

Apache License 2.0

2.9k stars 2.27k forks source link

Open awangc opened 1 month ago

awangc commented 1 month ago

exporter/elasticsearch

Currently in encoding a span, for Span Kind the exporter is adopting a string representation, but the Span Status adopts an integer representation (https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/exporter/elasticsearchexporter/model.go#L396-L397). It'd be great if we could adopt string representation for both, as it is more descriptive of the status.

github-actions[bot] commented 1 month ago

Pinging code owners:

exporter/elasticsearch: @JaredTan95 @ycombinator @carsonip

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

carsonip commented 1 month ago

This is the plan for OTel mapping mode as implemented in WIP PR: #34472

crobert-1 commented 1 month ago

Removing needs triage based on response from code owner.