pefernan commented on code in PR #3750:
URL:
https://github.com/apache/incubator-kie-kogito-runtimes/pull/3750#discussion_r1930788808
##########
addons/common/flyway/pom.xml:
##########
@@ -41,7 +41,10 @@
<groupId>org.flywaydb</groupId>
<artifactId>flyway-core</artifactId>
</dependency>
-
+ <dependency>
Review Comment:
@fjtirado the problem here is that flyway has changed a bit between v9 and
v10. In v9 the core module was including the support for almost all db like
pgsql, h2, mysql... but other propietary db'a (like sql server) had its own
`flyway-database-<vendor>` maven module that should be manually included in
the application pom.xml. In v10 they trimmed a bit more the flyway-core module,
leaving only H2 and SQL Lite. So to allow Flyway work with any specific DB that
isn't H2/SQL Lite it will be required adding the specific supporting database
module.
In this module we had to include the `flyway-database-postgresql` to allow
the tests pass.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
