Generator-jhipster: Change default database to Postgres

[agaspardcilia]

Overview of the feature request

As mentioned in #10958 we could use Postgres as our default database.

Motivation for or Use Case

The point must have been discussed without my knowledge to be in the v7 roadmap, so I'm making this ticket to have a place to talk about it.

Related issues or PR

I've already made some shallow stuff on #11724.

• [x] Checking this box is mandatory (this is just to show you read everything)

[agaspardcilia]

As @pascalgrimaud pointed out, changing this would require to:

• change all default config in CI to use and test more PostgreSQL
• do the same thing in jhipster-daily-builds
• change the behavior in jhipster-online
• do the change in all other blueprint backend (kotlin, nodejs, quarkus, etc.)

[MathieuAA]

And update what the JDL parsing system outputs to take that into account.

[mshima]

• Migrate to auto increment.
  https://github.com/liquibase/liquibase/releases/tag/v4.0.0-beta1
  https://github.com/liquibase/liquibase/releases/tag/v3.10.1

[DanielFran]

@mshima You mean "postgresql >= 10: use identity columns for autoincrement"?

[mshima]

@DanielFran maybe we should use autoIncrement feature for postgresql too.
https://github.com/jhipster/generator-jhipster/blob/902160664a79cce8130e8077867f52ef4c394eac/generators/entity-server/templates/src/main/resources/config/liquibase/changelog/added_entity.xml.ejs#L27-L33

[DanielFran]

@mshima Yes I agree, since I migrate to liquibase 3.10.1 (and PR is ready for 4.0.0), we can now switch to autoincrement.

Will you make the PR or do you want I take care of it?

[mshima]

@DanielFran posted to keep note, pretty busy in the generator refactoring.
Feel free to work on it.

[murdos]

Why would you want to use autoincrement for Postgres?
As far as I understand, the point in this article still stands: https://vladmihalcea.com/postgresql-serial-column-hibernate-identity/
The new identity feature in Postgres is just a more convenient and cleaner way to manage the underlying sequence used by Postgres, but this is still problematic with JPA/Hibernate.

[DanielFran]

@murdos The only changes expected is to use autoincrement in liquibase side that define the "GENERATED AS IDENTITY" that still uses a sequence

[pascalgrimaud]

Closing this as it's done