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

feat: support postgres wire protocol #62

Merged
merged 10 commits into from
Sep 28, 2023

Conversation

Lilit0x
Copy link
Contributor

@Lilit0x Lilit0x commented Sep 14, 2023

Which issue does this PR close?

Closes #44

What changes are included in this PR?

Adding support for PostgreSQL databases to run tests against

Are there any user-facing changes?

  • Multiple options for the type argument

How does this change test

CI

@CLAassistant
Copy link

CLAassistant commented Sep 14, 2023

CLA assistant check
All committers have signed the CLA.

@Lilit0x
Copy link
Contributor Author

Lilit0x commented Sep 14, 2023

@jiacai2050, I have the basic outline here, but I am quite stuck on how to integrate it with CLI through the EnvController seeing that it requires a DB type. And now, there are two types of DB.
So, I just repicated the EnvController trait for Postgres too and I know that's not the way to go, but I guess it is a starting point.

sqlness/src/database_impl/postgresql.rs Outdated Show resolved Hide resolved
sqlness-cli/src/main.rs Outdated Show resolved Hide resolved
@tanruixiang
Copy link
Contributor

@Lilit0x Hi, I've helped you make the appropriate changes based on the REVIEW, if you have time, please help with the REVIEW.

@jiacai2050 jiacai2050 changed the title Postgres Wire Protocol Support feat: support postgres wire protocol Sep 27, 2023
Copy link
Member

@jiacai2050 jiacai2050 left a comment

Choose a reason for hiding this comment

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

LGTM

@jiacai2050 jiacai2050 merged commit 8e1887e into CeresDB:main Sep 28, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support PostgreSQL wire protocol
4 participants