potiuk commented on issue #25104: URL: https://github.com/apache/airflow/issues/25104#issuecomment-1186104491
Agree it would be nice to improve it - feel free to create PR on it when we can iterate on possble solutions. I think it is important to keep backwards compatibility and do not go 2.3+. There must be a very good reason to break compatibility before the time, unless you want to wait till 11th October: https://github.com/apache/airflow#release-process-for-providers > For example this means that by default we upgrade the minimum version of Airflow supported by providers to 2.3.0 in the first Provider's release after 11th of October 2022 (11th of October 2021 is the date when the first PATCHLEVEL of 2.2 (2.2.0) has been released. Also I think there is another aspect here. We should make it in the way that: a) there is a parameter needed to determine if the parameters are url-encoded or not b) default should be true (but it should raise a deprecation warning). Otherwise we are risking that if someone configured it with URL-encoding (as cumbersome as it might see - you finally did it too) and if we just change the behaviour, we should not break experience of those people - otherwise upgrading AWS provider might mysteriously make their DAGs break. -- 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]
