eladkal edited a comment on issue #20139:
URL: https://github.com/apache/airflow/issues/20139#issuecomment-994017287


   > Batch has multiple Hook files, that may fall under this campaign? One is 
for the waiter and the other for the client though, so maybe not?
   
   Batch is not part of the action items of this issue as here we are focused 
on Operators/Sensors.
   Regardless my approach would be the same - We should follow what the user 
expects.
   If users expect to find the classes on the same file then we should merge - 
if the user doesn't expect it and the services are really distinct then we 
should keep separated. You might be interested in a (possibly) similar dilemma 
we had with redshift executing SQL and redshift managing the cluster:
   https://apache-airflow.slack.com/archives/CCPRP7943/p1639424289318500
   Eventualy we went with having redshift_sql and redshift_cluster as it's 
unlikely that the same ETL will do both cluster manipulation and executing sql. 
It's two distinct usages.
   


-- 
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]


Reply via email to