realtimetodie commented on code in PR #72458:
URL: https://github.com/apache/airflow/pull/72458#discussion_r3931003724
##########
airflow-core/src/airflow/config_templates/config.yml:
##########
@@ -1562,6 +1562,15 @@ traces:
type: integer
example: ~
default: "30000"
+ dag_tags_in_spans:
Review Comment:
In an upcoming pull request, I would like to change how DAG tags are
represented in OpenTelemetry.
Expanding every single tag into attributes is needed for StatsD-style
metrics, but OpenTelemetry supports string-array attributes. This means all DAG
tags in spans and metrics could be exported under a single attribute instead
for OpenTelemetry, for example:
`airflow.dag.tags = ["finance", "production", "team:data"]`
However, this would be a breaking change, which I tried to avoid here. I
think we should first establish consistent behaviour across spans and metrics.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
