airflow
airflow copied to clipboard
apache
Deprecation of AutoML services: Add deprecation warnings and raise exceptions for already deprecated ones
Before merging this PR we need to merge #38633
This PR is for raising exception for the deprecated services of AutoML (Tables, Vision, Video Intelligence, Natural Language) which are EoL or will be EoL soon.
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.
I think you need to rebase the PR. It's 52 commits behind and this is likely why it fails
@molcay -> can you please rebase/resolve conflict and respond/mark as resolved all conversations that you think were resolved?
cc: @VladaZakharova - there were some doubts for Google team about those deprecations - does this one look ok for you?
cc: @VladaZakharova - there were some doubts for Google team about those deprecations - does this one look ok for you?
Hi! Thanks, yes, changes look good, LGTM
@molcay -> can you please review and address/mark as resolved if they are - the comments made by @shahar1 ?
@potiuk, Ohh I see, there are some hidden conversations. I totally missed them. Sorry for the inconvenience. I will try to address and resolve them as soon as possible.
I will work on this test failure and send an update. Thank you for the review @shahar1 Also, thank you for pinging and guiding along the way @potiuk
Hey @shahar1 ! Would be nice if you can check the changes here again