molcay commented on PR #38633: URL: https://github.com/apache/airflow/pull/38633#issuecomment-2031668080
Hey @shahar1, Thank you again for your effort and struggles for this matter; very appreciated. --- > ...I didn't know that you've already created this PR, as you created it in Vlada's fork and not here :) Yes, I know I created [the PR ](https://github.com/VladaZakharova/airflow/pull/35)in Vlada's fork 😅 It was my fault for not mentioning the WIP PR. Sorry for the miscommunication. Vlada's fork is our first place to create PR and get internal review then we are creating the PR to this repo. When I was creating the PR in the Vlada's fork, I saw that there were (and still are, _since this is not merged yet_) some conflicts I put that comment to #38418. But couldn't revert (not before you at least 😅) Btw, I created the new PR: https://github.com/apache/airflow/pull/38673 --- > After merging the current PR - you may recreate the PR here, I'll close mine and review yours ... If we can do this, it will be wonderful because for [the PR](https://github.com/apache/airflow/pull/38673) I tried to comply with the [Airflow Deprecation Policy](https://airflow.apache.org/docs/apache-airflow/stable/release-process.html#deprecation-policy). > **e.g:** Giving alternative operators instead of the deprecated and/or shutdown services' operators Also, we will continue to support most of the operators (at least for the next major version of the package) due to the deprecation policy. They will definitely fail and we will raise an exception for the users who are using the operators but it needs to be there. Hence we cannot remove the system tests. Also, the main idea of the new PR is finding alternatives/replacements of the operators for the deprecated services instead of just removing them. --- To summarize the current situation, We have 4 PRs: | PR | Summary | |--------|-----------------------------------------------------------| | #38418 | Merged and need to be reverted | | #38633 | This PR reverting #38418 | | #38635 | The other PR related to this topic | | #38673 | This is the new PR that contains policy complaint changes | -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
