uranusjr commented on code in PR #37725:
URL: https://github.com/apache/airflow/pull/37725#discussion_r1538855386
##########
airflow/providers/openlineage/utils/utils.py:
##########
@@ -73,6 +78,16 @@ def is_operator_disabled(operator: BaseOperator |
MappedOperator) -> bool:
return get_fully_qualified_class_name(operator) in
conf.disabled_operators()
+def is_selective_lineage_enabled(obj) -> bool:
+ """If selective enable is active check if DAG or Task is enabled to emit
events."""
+ if not conf.selective_enable():
+ return True
Review Comment:
Is this correct? This function seems to return True when lineage is enabled,
but this branch returns True if `selective_enable` returns False.
--
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-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
