ac1997 opened a new pull request #21351: URL: https://github.com/apache/airflow/pull/21351
Replacing existing celery subclass check in celery cli validation with a new attribute to Celery based executors. Also moved celery module check a level higher. IMO, it is worth checking even if the user is using built-in Celery or C-K executors. The reason behind this change is to allow any custom executor to utilize built-in airflow celery executor/worker. There are some cases that doesn't make sense to subclass the existing Celery / C-K executors (for example a similar implementation of C-K executor that has some other executors for routing, but not Kubernetes". This change should be backward compatible, assuming subclasses of C/C-K in the wild does not have `supports_celery` defined for other use. -- 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]
