uranusjr commented on PR #70280: URL: https://github.com/apache/airflow/pull/70280#issuecomment-5222809674
I don’t think this is going to fly either, at least as-is. `ExecuteTask.make()` is used in providers, you need to add this to more places. Unfortunately, providers also introduce complex version compatibility issues, so you’ll need to figure out a way to ensure things work correctly between different version combinations. And for the combinations where things can’t work, you’ll need to be able to do error handling, preferrably at parse time. The two easiest ways this can go down are either to silently make the argument do nothing (or drop it with a warning), or make the scheduler crash. Neither is acceptable IMO. -- 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]
