You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/08/02 21:10:38 UTC

[GitHub] [airflow] potiuk commented on a diff in pull request #25413: Refactor monolithic ECS Operator into Operators, Sensors, and a Hook

potiuk commented on code in PR #25413:
URL: https://github.com/apache/airflow/pull/25413#discussion_r936030041


##########
airflow/contrib/operators/ecs_operator.py:
##########
@@ -1,30 +0,0 @@
-#

Review Comment:
   I have an idea for the future. Should we move alll the deprecated/contrib operators from 1.10 out to a separate "apache-contrib" package (when we split out providers). 
   
   That will allow to get rid of the contrib, and get rid of many 1.10 deprecations out of airlfow main repo at least (it can still be a dependency of Airflow package so that it is installed automatically when airflow is installed). @ashb  @kaxil WDYT?



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org