feluelle commented on a change in pull request #12096:
URL: https://github.com/apache/airflow/pull/12096#discussion_r553873793



##########
File path: UPDATING.md
##########
@@ -742,6 +742,7 @@ The following operators were affected:
 * BashOperator (Not backwards compatible)
 * DockerOperator (Not backwards compatible)
 * SimpleHttpOperator (Not backwards compatible)
+* ECSOperator (Not backwards compatible)

Review comment:
       Oh @baxievski I think if we put this line here it will be confusing 
since version 2.0.0 has already been released. And with this in mind if you 
would add it to the next release > 2.0.0, it is no longer a breaking change, is 
it?
   
   So I would suggest we remove it completely.
   

##########
File path: UPDATING.md
##########
@@ -742,6 +742,7 @@ The following operators were affected:
 * BashOperator (Not backwards compatible)
 * DockerOperator (Not backwards compatible)
 * SimpleHttpOperator (Not backwards compatible)
+* ECSOperator (Not backwards compatible)

Review comment:
       Oh @baxievski I think if we put this line here it will be confusing 
since version 2.0.0 has already been released. And with this in mind if you 
would add it to the next release > 2.0.0, it is no longer a breaking change, is 
it?
   
   So I would suggest we remove it completely.
   
   ```suggestion
   ```




----------------------------------------------------------------
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]


Reply via email to