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.

Sign up for GitHub

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

Jump to bottom

refactor(risingwave): decouple the risingwave backend from the postgres backend #10669

Merged
merged 1 commit into from
Jan 15, 2025
Merged

refactor(risingwave): decouple the risingwave backend from the postgres backend #10669

merged 1 commit into from
Jan 15, 2025

Conversation

cpcloud
Copy link
Member

@cpcloud cpcloud commented Jan 15, 2025

Decouple the risingwave backend from the postgres backend to allow use of psycopg (not psycopg2) with the postgres backend in #10659.

@cpcloud cpcloud added this to the 10.0 milestone Jan 15, 2025
@cpcloud cpcloud added refactor Issues or PRs related to refactoring the codebase risingwave The RisingWave backend labels Jan 15, 2025
gforsyth
gforsyth approved these changes Jan 15, 2025
View reviewed changes
Copy link
Member

@gforsyth gforsyth left a comment

Choose a reason for hiding this comment

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

:shipit:

@gforsyth gforsyth merged commit bd30ba5 into ibis-project:main Jan 15, 2025
91 checks passed
@cpcloud cpcloud mentioned this pull request Jan 17, 2025
Merged
@cpcloud cpcloud deleted the decouple-risingwave-from-postgres-backend branch January 21, 2025 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gforsyth gforsyth gforsyth approved these changes

Assignees
No one assigned
Labels
refactor Issues or PRs related to refactoring the codebase risingwave The RisingWave backend
Projects
None yet
Milestone
10.0
Development

Successfully merging this pull request may close these issues.
2 participants
@cpcloud @gforsyth