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

Add AWS Redshift Serverless support to PostgresHook #43669

Merged

Conversation

topherinternational
Copy link
Contributor

@topherinternational topherinternational commented Nov 5, 2024

This change add AWS Redshift Serverless support to the PostgresHook.

PostgresHook now supports the following keys in the extras of the connection parameters (or the query parameters if the connection is specified by a URI):

  • redshift-serverless: set this to True to authenticate and connect to an AWS Redshift Serverless authentication
  • workgroup-name: the name of the workgroup associated with the database, to be passed to the redshift-serverless client's get_credentials method when authenticating (if the workgroup-name is not provided, the hook falls back to the first label of the hostname in the same fashion as the redshift option for PostgresHook)

Copy link
Contributor

@vincbeck vincbeck left a comment

Choose a reason for hiding this comment

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

LGTM

@topherinternational
Copy link
Contributor Author

To most of the reviewers - your being requested was an accident, a rebase-push went bad and brought in a bunch of other changes so a bunch of people got tagged.

@eladkal eladkal merged commit 43f3c31 into apache:main Nov 6, 2024
56 checks passed
ellisms pushed a commit to ellisms/airflow that referenced this pull request Nov 13, 2024
* add AWS Redshift Serverless support to PostgresHook

* exception types and doc strings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants