ashb commented on issue #6230: [AIRFLOW-5413] Allow K8S worker pod to be 
configured from JSON/YAML file
URL: https://github.com/apache/airflow/pull/6230#issuecomment-573254758
 
 
   > That's a tough one as that would break a LOT of DAGs, that said it would 
be nice to have some class typing as it's hard to know off-hand what configs 
you can set.
   > 
   > @ashb @potiuk @nuclearpinguin @kaxil What do you guys think about this? 
Maybe we have an `executor_config_dict` argument for easy backporting and 
slowly deprecate?
   
   It should be doable to detect a dict and "upgrade" it in the same argument. 
It would be nice if we supported both for a few releases - we _could_ just say 
F-it and need an update for Airflow 2.0, but I'd like us to avoid any hard 
breaking changes to DAGs without a very good reason (and us regretting our past 
code is not a good enough reason for me :smile: )

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to