xBis7 commented on code in PR #63932:
URL: https://github.com/apache/airflow/pull/63932#discussion_r3124956768
##########
providers/openlineage/src/airflow/providers/openlineage/plugins/listener.py:
##########
@@ -204,7 +205,13 @@ def on_running():
if not doc:
doc, doc_type = get_dag_documentation(dag)
- with Stats.timer(f"ol.extract.{event_type}.{operator_name}"):
+ if AIRFLOW_V_3_2_1_PLUS:
+ ctx = Stats.timer(
+ "ol.extract", legacy_name_tags={"event_type": event_type,
"operator_name": operator_name}
+ )
+ else:
+ ctx = Stats.timer(f"ol.extract.{event_type}.{operator_name}")
+ with ctx:
Review Comment:
Same here, I simplified the approach and we don't need any back-compat
checks anymore. Everything is handled internally by stats and the public
interface stays the same.
--
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]
