feat: durable state database support

[Roiocam]

Motivation

resolves: #56

tracker

• MySQL support: MySQL has very different ways to generate auto increment column
• Oracle support
• SqlServer support
• update documentation

[laglangyue]

how to test it,is there any integration testing?

[pjfanning]

Would it be possible to add string constants for all these Strings?

      case PostgresProfile  => "Postgres"
      case MySQLProfile     => "MySQL"
      case OracleProfile    => "Oracle"
      case SQLServerProfile => "SqlServer"
      case H2Profile        => "H2"

And then to remove the direct usage of the strings - use the constants instead.

[mdedetrich]

Would it be possible to add string constants for all these Strings?

      case PostgresProfile  => "Postgres"
      case MySQLProfile     => "MySQL"
      case OracleProfile    => "Oracle"
      case SQLServerProfile => "SqlServer"
      case H2Profile        => "H2"

And then to remove the direct usage of the strings - use the constants instead.

Definite +1 from me

[Roiocam]

It may need another separate PR for MySQL support.

[pjfanning]

Unfortunately, the Oracle integration tests are failing due to table name issues.

[Roiocam]

Unfortunately, the Oracle integration tests are failing due to table name issues.

Seems like latest commit break this, I will figure out later

[Roiocam]

Unfortunately, the Oracle integration tests are failing due to table name issues.

fixed

[pjfanning]

@Roiocam can you create an issue for following up with MySQL support - so at least, the issue is there for other users to examine and maybe provide a solution?

[Roiocam]

@Roiocam can you create an issue for following up with MySQL support - so at least, the issue is there for other users to examine and maybe provide a solution?

#174 created.

[Roiocam]

All suggestions have been applied, there are the changes: https://github.com/apache/pekko-persistence-jdbc/pull/158/files/1d3b1e5bf22ac6f7a3cf8c68fd0ad883f3d7bd84..c7bc649641a01c9471754852398d4f44299d9023

[pjfanning]

lgtm