[GitHub] [airflow] potiuk commented on pull request #28961: Emit DataDog statsd metrics with metadata tags

["potiuk (via GitHub)" <gi...@apache.org>]

potiuk commented on PR #28961:
URL: https://github.com/apache/airflow/pull/28961#issuecomment-1435655836

   Yes. I think if we see high-cardinality metrics we could add features to disable them indeed - not sure though if it should be done a single "disable-high-cardinality" metrics or list of metrics to disable. Both have advantages and disadvantages. I think the single flag is more opinionated what is high-cardinality, but it has also the potential on being used in OTEL implementaiton (cc: @feruzzi). 
   
   It looks like the discussion on what is/should be cardinality explanation and making the documentation and explanation of it part of the OTEL specification https://github.com/open-telemetry/opentelemetry-specification/issues/2996 so once we get into the OTEL implementation we could also think about it and take part in the discussion.
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org