sunank200 commented on code in PR #46231:
URL: https://github.com/apache/airflow/pull/46231#discussion_r1936814566
##########
tests_common/pytest_plugin.py:
##########
@@ -40,7 +40,7 @@
from airflow.models.dag import DAG, ScheduleArg
from airflow.models.dagrun import DagRun, DagRunType
from airflow.models.taskinstance import TaskInstance
- from airflow.operators.empty import EmptyOperator
+ from airflow.providers.standard.operators.empty import EmptyOperator
Review Comment:
why import it here if it is imported lazily in the file inside functions?
##########
tests/serialization/test_dag_serialization.py:
##########
@@ -64,9 +64,9 @@
from airflow.models.expandinput import EXPAND_INPUT_EMPTY
from airflow.models.mappedoperator import MappedOperator
from airflow.models.xcom import XCom
-from airflow.operators.empty import EmptyOperator
from airflow.providers.cncf.kubernetes.pod_generator import PodGenerator
from airflow.providers.standard.operators.bash import BashOperator
+from airflow.providers.standard.operators.empty import EmptyOperator
Review Comment:
why import it here if it is imported lazily in the file inside functions?
--
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]
