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

[CT-1504] [Feature] Replace psycopg2 with redshift_connector #219

Closed
3 tasks done
dbeatty10 opened this issue Nov 14, 2022 · 1 comment · Fixed by #251
Closed
3 tasks done

[CT-1504] [Feature] Replace psycopg2 with redshift_connector #219

dbeatty10 opened this issue Nov 14, 2022 · 1 comment · Fixed by #251
Labels
enhancement New feature or request

Comments

@dbeatty10
Copy link
Contributor

dbeatty10 commented Nov 14, 2022

Is this your first time submitting a feature request?

  • I have read the expectations for open source contributors
  • I have searched the existing issues, and I could not find an existing issue for this feature
  • I am requesting a straightforward extension of existing dbt-redshift functionality, rather than a Big Idea better suited to a discussion

Describe the feature

Currently, dbt-redshift inherits the DB API 2.0 Python connector from dbt-postgres (which defaults to psycopg2-binary).

redshift_connector is the Amazon Redshift connector for Python that implements Python Database API Specification 2.0 (PEP 249).

Implementing redshift_connector within dbt-redshift may unlock or ease other open issues like:

Describe alternatives you've considered

Leaving the psycopg2 is an option, but it might not support Redshift-specific functionality like mentioned above.

Who will this benefit?

No response

Are you interested in contributing this feature?

No response

Anything else?

This was discussed in a video call with @jtcohen6 and @Fleid earlier today, and it was affirmed that it might be time to take on this implementation.

@dbeatty10 dbeatty10 added enhancement New feature or request triage labels Nov 14, 2022
@github-actions github-actions bot changed the title [Feature] Replace psycopg2 with redshift_connector [CT-1504] [Feature] Replace psycopg2 with redshift_connector Nov 14, 2022
@dbeatty10 dbeatty10 removed the triage label Nov 15, 2022
@iklose
Copy link

iklose commented Nov 17, 2022

This could also solve problems regarding Redshift Serverless connections with IAM authentication: #138

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants