airflow icon indicating copy to clipboard operation
airflow copied to clipboard

removing deprecation postgres

Open Bowrna opened this issue 1 year ago • 1 comments
trafficstars

Related: https://github.com/apache/airflow/issues/39485 Removing deprecated PostgresOperator from docs and tests and replacing with SQLExecuteQueryOperator


^ 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.

Bowrna avatar May 20 '24 06:05 Bowrna

Looks really nice!

I would suggest to keep information about PostgresOperator into the documentation, e.g. in warning block This might help users who try to search PostgresOperator via search engines (e.g. in google). In this case, potentially, it come to the current version documentation page, otherwise it might show old version of provider.

Something like that


.. warning::
    Previously, PostgresOperator was used to perform this kind of operation. But at the moment PostgresOperator is deprecated and will be removed in future versions of the provider. Please consider to switch to SQLExecuteQueryOperator as soon as possible.

Taragolis avatar May 20 '24 09:05 Taragolis

@Taragolis I have added the warning part in docs. Let me know if it looks good now. Will do the same for the other DB related providers too!

Bowrna avatar May 21 '24 05:05 Bowrna