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 2021/07/12 23:16:22 UTC

[GitHub] [airflow] ferruzzi edited a comment on pull request #16571: Implemented Basic EKS Integration

ferruzzi edited a comment on pull request #16571:
URL: https://github.com/apache/airflow/pull/16571#issuecomment-878659314


   > It turns out that writing the EKSPodOperator operator correctly is not trivial. What do you think about contributing it as a separate PR?
   
   I'm not against doing this in multiple PRs.  My primary concern is that without the EKSPodOperator, the other Operators and Hooks are not all that useful.  I'm not a huge fan of adding a chunk of "useless" code on the promise that "it will make sense later", but if you are good with it in this case (and presumably you are or you wouldn't have suggested it) then I can live with 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