adutra commented on PR #1695: URL: https://github.com/apache/polaris/pull/1695#issuecomment-2988259372
> > > > Optional follow-up: merge polaris-server into polaris-service to simplify the module layout (not a blocker for this PR). > > > > > > > > > I'm strongly -1 on this. I'm pretty sure this is going to complicate the reuse of `polaris-service` code in downstream projects. > > > > > > Agreed. However, in a follow-up PR, imho we could merge `polaris-service-common` into `polaris-runtime-service`. > > I agree to have a follow-up PR and happy to do that. I think the most important for 1.0 release is about the Maven coordinates and names "visible" to users. @jbonofre do you plan to work on this? Otherwise I can look into it. While it's not so critical, it's still a user-facing change since the `polaris-service-common` artifact would disappear. -- 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]
