Taragolis commented on code in PR #29001:
URL: https://github.com/apache/airflow/pull/29001#discussion_r1073444102
##########
airflow/providers/amazon/aws/sensors/batch.py:
##########
@@ -74,16 +73,13 @@ def poke(self, context: Context) -> bool:
raise AirflowException(f"Batch sensor failed. Unknown AWS Batch job
status: {state}")
Review Comment:
One use case it create custom Operator/Sensor which based on existed
Sensor/Operator and overwrite particular method.
So better to keep existed public method at least for a while.
And we need to start remove deprecated part again in next major release.
--
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]
