adutra opened a new pull request, #628: URL: https://github.com/apache/polaris/pull/628
⚠️ Not ready for review yet! Requires #469 and #604. This PR splits `polaris-quarkus-service` in two modules: * `polaris-quarkus-service` contains all the Quarkus-specific components needed for the server, and soon, for the admin tool as well. * `polaris-quarkus-server` builds the final artifact (the runnable server). It contains the server's `application.properties` and the `Dockerfile.jvm` to build the docker image. This PR is mainly motivated by the fact that there cannot be more than one `application.properties` file in the classpath. Keeping it in `polaris-quarkus-service` would make it impossible for the future admin tool to have its own configuration file while depending on `polaris-quarkus-service`. The common parts of the configuration were moved to `META-INF/microprofile-config.properties` in `polaris-quarkus-service`. This PR also fixes the server binary distribution by adapting it to Quarkus. -- 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]
