You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/12/09 21:37:59 UTC

[GitHub] [airflow] pierrejeambrun commented on pull request #28273: Make CeleryKubernetesExecutor extends BaseExecutor

pierrejeambrun commented on PR #28273:
URL: https://github.com/apache/airflow/pull/28273#issuecomment-1344805905

   APIs between those 2 are too different, and would require further effort to have it inherit from BaseOperator. (Without having to ignore overrides, skip parent constructor, which could lead to future issue). I think we would need to use properties everywhere so each subclass has better control on how he want to handle it.


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org