potiuk commented on code in PR #41493: URL: https://github.com/apache/airflow/pull/41493#discussion_r1732717474
########## newsfragments/41493.significant.rst: ########## @@ -0,0 +1 @@ +Removed deprecated method ``task`` from ``airflow.operators.python`` operator. Please use ``python_task`` from ``airflow.decorators.python`` instead. Review Comment: I think we will have to keep it in newsfragments, so far those were "core" operators - and we have not moved them to "provider.standard" yet - and as I understand that, the standard provider will be part of the Airflow 3 migration (we will not remove old Python operators from Airflow 2 - we will add deprecation there) so the newsfragment should be sill in Airflow - I think. ########## newsfragments/41493.significant.rst: ########## @@ -0,0 +1 @@ +Removed deprecated method ``task`` from ``airflow.operators.python`` operator. Please use ``python_task`` from ``airflow.decorators.python`` instead. Review Comment: I think we will have to keep it in newsfragments, so far those were "core" operators - and we have not moved them to "provider.standard" yet - and as I understand that, the standard provider will be part of the Airflow 3 migration (we will not remove old Python operators from Airflow 2 - we will add deprecation there) so the newsfragment should be still in Airflow - I think. -- 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]
