[GitHub] [airflow] bdsoha commented on a diff in pull request #27974: [AIP-51] Add helper to import default executor class
bdsoha commented on code in PR #27974: URL: https://github.com/apache/airflow/pull/27974#discussion_r1034335803 ## airflow/executors/executor_loader.py: ## @@ -65,18 +65,23 @@ class ExecutorLoader: DEBUG_EXECUTOR: "airflow.executors.debug_executor.DebugExecutor", } +@classmethod +def get_default_executor_name(cls) -> str: +"""Returns the default executor name from Airflow configuration Review Comment: ```suggestion """Return the default executor name from Airflow configuration. ``` -- 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
[GitHub] [airflow] bdsoha commented on a diff in pull request #27974: [AIP-51] Add helper to import default executor class
bdsoha commented on code in PR #27974: URL: https://github.com/apache/airflow/pull/27974#discussion_r1034335803 ## airflow/executors/executor_loader.py: ## @@ -65,18 +65,23 @@ class ExecutorLoader: DEBUG_EXECUTOR: "airflow.executors.debug_executor.DebugExecutor", } +@classmethod +def get_default_executor_name(cls) -> str: +"""Returns the default executor name from Airflow configuration Review Comment: ```suggestion """Returns the default executor name from Airflow configuration. ``` -- 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