Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Redshift mention from the list of managed Postgres backends #36217

Merged

Conversation

Taragolis
Copy link
Contributor

Redshift is mentioned as managed Postgres but is far away from the reality better to say it is product based on very old Postgres (something like 8.4) or even maybe only use Postgres protocol nowadays, difficult to say because it is cloud only proprietarily product.

Anyway there is small chance that end users might run Airflow by use Redshift as backend DB, so better remove from documentation fir prevent any confusion and misleading.


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

Copy link
Member

@hussein-awala hussein-awala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree, AWS used Postgres when they created Redshift, but I don't think it's the case today.

@potiuk
Copy link
Member

potiuk commented Dec 14, 2023

Agree. This is "Redshift" and if people will use it at "Postgres Metadata" - weird things will happen. Like airlfow will sort-of works, which is worst outcome.

@vincbeck vincbeck merged commit 01fd0d3 into apache:main Dec 14, 2023
49 checks passed
@ephraimbuddy ephraimbuddy added this to the Airflow 2.8.1 milestone Jan 10, 2024
@ephraimbuddy ephraimbuddy added the type:doc-only Changelog: Doc Only label Jan 10, 2024
ephraimbuddy pushed a commit that referenced this pull request Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants