dimberman 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-573262847 > > 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 😄 ) ooo I like that idea. We can do type checking/deprecation warnings for the first few versions and slowly wean people off without needing extraneous parameters.
---------------------------------------------------------------- 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
