potiuk commented on code in PR #25413: URL: https://github.com/apache/airflow/pull/25413#discussion_r937572740
########## airflow/contrib/operators/ecs_operator.py: ########## @@ -1,30 +0,0 @@ -# Review Comment: Well. They shoudl be. But this is not a must actually. Contrib were part of Airflow 1.10. When we moved to 2, we vowed to keep MOSTLY compatibility with 1.10. I.e. people who still used 1.10 would be able to continue using them. There were a number of exceptions and back then we had "backport" providers so people could simply move to use backports "airfow.providers.*" but we stoppped (as planned) releasing those, so the "late-comers" will have to deal with conesequences ot potentially broken compatibility if they still use contrib. Since then many of those Contrib operators and hooks were modified and they are not even backwards-compatible with the original Contrib ones. So technically speaking, if an operator is anyhow backwards incompatible we do not have to keep it in contrib. But if it largely is, it makes it easier for user to have smooth migration. -- 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]
