This repository was archived by the owner on Jan 19, 2022. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 691
postgresql support for spring-cloud-gcp-data-jpa-sample #401
Labels
Comments
We should autoconfigure the dialect. It will help with #392 too. |
Is there anything we want to do here? Do we want to have a line commented out next to this line with the PostgreSQL dialect? |
I think one way to resolve this issue is to add a section on the Cloud SQL sample about how to change it to work with PostgreSQL. |
I think the following configurations should be added to work on this sample with PostgreSQL properly. #Disable feature detection by this undocumented parameter. #Because detection is disabled you have to set correct dialect by hand. |
@mkatircioglu Thanks for the feedback on the issue! |
meltsufin
added a commit
that referenced
this issue
Apr 1, 2019
Added commented out code and documentation in the sample `README.md`. Fixes: #401.
meltsufin
added a commit
that referenced
this issue
Apr 1, 2019
Added commented out code and documentation in the sample `README.md`. Fixes: #401.
elefeint
pushed a commit
that referenced
this issue
May 28, 2021
Bumps [libraries-bom](https://github.com/GoogleCloudPlatform/cloud-opensource-java) from 19.2.0 to 19.2.1. - [Release notes](https://github.com/GoogleCloudPlatform/cloud-opensource-java/releases) - [Changelog](https://github.com/GoogleCloudPlatform/cloud-opensource-java/blob/master/CHANGELOG.md) - [Commits](https://github.com/GoogleCloudPlatform/cloud-opensource-java/commits) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The JPA sample doesn't state that it's only for MySQL instances, so I tried to run it with postgresql.
Here are the things I needed to modify:
application.properties:
pom.xml: add the postgresql starter instead of the mysql one
After the above, it works!
During startup I still get a warning, but I guess that can be ignored:
The text was updated successfully, but these errors were encountered: