-
Notifications
You must be signed in to change notification settings - Fork 119
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
Nessie server: add support for MariaDB and MySQL backends #8544
Conversation
129f697
to
82f28d8
Compare
...storage/jdbc/src/main/java/org/projectnessie/versioned/storage/jdbc/AbstractJdbcPersist.java
Outdated
Show resolved
Hide resolved
...d/storage/jdbc/src/main/java/org/projectnessie/versioned/storage/jdbc/DatabaseSpecifics.java
Show resolved
Hide resolved
82f28d8
to
619cfef
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like a bunch of unrelated changes made it into this PR. Can you move these to a separate one?
servers/quarkus-server/src/main/resources/application.properties
Outdated
Show resolved
Hide resolved
servers/quarkus-common/src/main/java/org/projectnessie/quarkus/config/QuarkusJdbcConfig.java
Show resolved
Hide resolved
...main/java/org/projectnessie/quarkus/tests/profiles/BigTableTestResourceLifecycleManager.java
Outdated
Show resolved
Hide resolved
Which changes are unrelated? All the changes in the persist layer were done in order to optimize the configuration of Quarkus tests, by introducing the new The only change that is a bit unrelated indeed is the one I mentioned here: I will move that one to another PR. |
560f19e
to
a9ebc42
Compare
a9ebc42
to
011d87e
Compare
Ready for review again! |
...src/main/java/org/projectnessie/quarkus/tests/profiles/QuarkusTestProfilePersistMariaDB.java
Show resolved
Hide resolved
servers/quarkus-common/src/main/java/org/projectnessie/quarkus/config/QuarkusJdbcConfig.java
Outdated
Show resolved
Hide resolved
...us-common/src/main/java/org/projectnessie/quarkus/config/datasource/DataSourceActivator.java
Outdated
Show resolved
Hide resolved
...us-common/src/main/java/org/projectnessie/quarkus/config/datasource/DataSourceActivator.java
Outdated
Show resolved
Hide resolved
...us-common/src/main/java/org/projectnessie/quarkus/config/datasource/DataSourceActivator.java
Outdated
Show resolved
Hide resolved
...us-common/src/main/java/org/projectnessie/quarkus/config/datasource/DataSourceActivator.java
Outdated
Show resolved
Hide resolved
...us-common/src/main/java/org/projectnessie/quarkus/config/datasource/DataSourceActivator.java
Outdated
Show resolved
Hide resolved
...us-common/src/main/java/org/projectnessie/quarkus/config/datasource/DataSourceActivator.java
Show resolved
Hide resolved
...kus-common/src/main/java/org/projectnessie/quarkus/providers/storage/JdbcBackendBuilder.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
But let's wait for "all the MariaDB" PRs to be ready and merge those "together".
No description provided.