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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
