potiuk commented on code in PR #32526:
URL: https://github.com/apache/airflow/pull/32526#discussion_r1260106204


##########
tests/sensors/test_base.py:
##########
@@ -44,6 +42,8 @@
 from airflow.models import TaskReschedule
 from airflow.models.xcom import XCom
 from airflow.operators.empty import EmptyOperator
+from airflow.providers.celery.executors.celery_executor import CeleryExecutor
+from airflow.providers.celery.executors.celery_kubernetes_executor import 
CeleryKubernetesExecutor

Review Comment:
   Not in core, but we still do in tests. I do not **yet** want to disentangle 
those - because some of our tests in Airflow actually use either Celery or 
Kubernetes executor. Those are not necessary to be removed from core **just 
yet**. We will deal with it later if/when we actually split providers more.



-- 
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]

Reply via email to