dimas-b opened a new issue, #3569:
URL: https://github.com/apache/polaris/issues/3569
### Is your feature request related to a problem? Please describe.
This is a proposal to refactor inter-module dependencies in order to allow
easier selection of pluggable features in downstream builds.
* Move pluggable (optional) module dependencies out of `runtime/services`
into `runtime/server`
* Move the majority of integration tests into new sub-modules under the
pluggable components
- Example: `persistence/relational-jdbc/tests`
- Prior art: `extensions/auth/opa/tests`
* Run "smoke tests" for the default set of plugins under the
`runtime/server` module (e.g. validating PostgreSQL integration, but without
extensive code coverage, which will be done under
`persistence/relational-jdbc/tests`).
The functionality provided by the `runtime/server` module will not be
altered by this change.
Pros:
* It will be easier to select a sub-set of optional features to include in
downstream builds (some of which may not need all of the pluggable features)
* The `runtime/service` module becomes lean and focused on Polaris
fundamentals.
Cons:
* CI time may increase due to having to perform more Quarkus builds for
integration tests (might be mitigated by parallel execution).
### Describe the solution you'd like
_No response_
### Describe alternatives you've considered
_No response_
### Additional context
_No response_
--
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]
