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

Doltgresql does not yet support PostgreSQL's JDBC connection #279

Closed
Ambitie opened this issue May 16, 2024 · 7 comments
Closed

Doltgresql does not yet support PostgreSQL's JDBC connection #279

Ambitie opened this issue May 16, 2024 · 7 comments

Comments

@Ambitie
Copy link

Ambitie commented May 16, 2024

Doltgresql does not yet support PostgreSQL's JDBC connection

I can only connect to the doltgresql database after setting assumeMinServerVersion=15. But after connecting to the doltgresql database, I cannot execute any queries.

@zachmu
Copy link
Member

zachmu commented May 16, 2024

Thanks for the report @Ambitie. We're looking into this and should have an estimate for when you can expect this work in the near future.

@jennifersp
Copy link
Contributor

Hi @Ambitie, I just got a PR out that might fix this issue here. Can you try testing with this change?

@timsehn
Copy link
Collaborator

timsehn commented May 16, 2024

Testing will require building from source: https://github.com/dolthub/doltgresql/blob/main/CONTRIBUTING.md#getting-set-up

@zachmu
Copy link
Member

zachmu commented May 16, 2024

If you don't want to build from source, can you confirm the error you were getting looked like this?

org.postgresql.util.PSQLException: A result was returned when none was expected.

example.zip

I have a simple repro that fails with this error before Jennifer's change and passes after.

@zachmu
Copy link
Member

zachmu commented May 20, 2024

@Ambitie we are pretty sure this is fixed in release 0.7.5, which is going out now.

To close this bug out, we need an automated client test of the JDBC functionality, similar to what Dolt has.

@bpf120
Copy link

bpf120 commented May 22, 2024

@Ambitie , thanks again for filing this. We'd love to learn about your Doltgres use case. You can email me or swing by our Discord if you'd like to share.

@jennifersp
Copy link
Contributor

Closing the issue as a test for it has been added #297

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

No branches or pull requests

5 participants