dstandish commented on a change in pull request #13247:
URL: https://github.com/apache/airflow/pull/13247#discussion_r552081114
##########
File path: airflow/cli/cli_parser.py
##########
@@ -30,7 +30,6 @@
from airflow.cli.commands.legacy_commands import check_legacy_command
from airflow.configuration import conf
from airflow.exceptions import AirflowException
-from airflow.executors.executor_constants import CELERY_EXECUTOR,
CELERY_KUBERNETES_EXECUTOR
Review comment:
for context, in [a previous
change](https://github.com/apache/airflow/commit/d752d92a4d8ef0dee93519397e76c1ecb0eda548#diff-521fc27e2c47783cd582bbe615daeba4b6d10eee86f7f923b72e5adbc15f8eb1),
when i thought that CeleryKubernetesExecutor should be consistently set across
the cluster, i had added one of them.
in _this_ pr though i was essentially reverting that change.
in my memory the constants were not use prior to my change, but in
actuality, the constant was used prior to my prev change -- in the if statement
but not in the logging. so it is actually a removal. and i will restore use
of constant.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
