dstandish commented on pull request #19572: URL: https://github.com/apache/airflow/pull/19572#issuecomment-985812094
>> @jedcunningham one thing i have not given a ton of thought is what should go in PodLauncher vs what should go in the operator. do you think that's something that i need to look into? > Yeah, probably worth giving that some thought while we are at it. yeah i took another look and gave it some more thought. i think it's ok. i think PodLauncher is sort of an instance of a kind of "manager" design pattern which just helps to make the operator more about orchestration, more higher level, and can make it easier to write tests sometimes. it's like equivalent to a hook but specific to this operator. we could shuffle things around but i'm not sure there is much value to be gained there. let me know if you think otherwise. -- 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]
