Taragolis commented on PR #29522:
URL: https://github.com/apache/airflow/pull/29522#issuecomment-1435483536

   As I mention before I do not not have strict concern about is it should be 
single operator or 3 operators (Regular, Node, EKS). I use combination Airflow 
+ Batch since Sept 2019, and this combination cover a lot of limitations of 
each other. Like same implementation as Dynamic Task Mapping available in Batch 
years ago and work thought 
[arrayProperties](https://docs.aws.amazon.com/batch/latest/userguide/array_jobs.html),
 in the other side dependency between Batch Jobs not such good rather Airflow 
   
   And there is no many changes happen in Batch operator since this time 
however the design of Hooks and BatchOperator still from pre-provider era and 
now it looks ugly even if it have exclusive backoff API caller. 
   
   Moth concern that potentially most of the parameters exclusively for 
containerOverrides options and we do not check it right now. I'm not a user of 
nodeOverrides because usually such architecture more suits for Hadoop Cluster, 
so for that purpose better to use EMR. Different users different point.
   
   I would try to check that options and return back after weekend.
   


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