Hi Alex, It sounds good. And I agree.
I'm happy to include Quarkus merge for 0.9.0 (rc2), that makes perfect sense to me and open the path for the next releases without "user facing change". That said, it means I have to do a new pass on the LICENSE (especially for the binary distributions) and NOTICE. As we are not in the rush about rc2 (we agreed to do rc2 in the coming weeks). Let's just give the time to the community to be back from "festive break". Regards JB On Tue, Dec 31, 2024 at 2:32 PM Alex Dutra <alex.du...@dremio.com.invalid> wrote: > > Hi all, > > I hope everyone is having a great holiday season so far! > > Unless I'm mistaken, I think we have an agreement to move forward with the > switch to Quarkus. I'm wondering when we should merge it? Keeping such a large > PR open for too long is not ideal, but I also don't want to rush it. > > Here 's some useful context to help us decide: > > The PR has been up for review for a few weeks now, and I encourage everyone to > take a look at it: : https://github.com/apache/polaris/pull/469. > > See also the proposal document: > > https://docs.google.com/document/d/1C7E0ma6OvHGvWMJlZVkQHBsvPg8Pe9aroDtQ_79SIXY/edit > > Luckily, the changes are not too invasive for most modules – except for the > Dropwizard runtime module, of course. People can continue working on modules > like core, api, persistence, service-common, etc. without much trouble. > > OTOH, merging soon-ish would settle the dust, and we could start focusing more > on the new features that we have planned for the next releases. > > Merging would also unlock a few other PRs that are waiting on the Quarkus > switch: > > - https://github.com/apache/polaris/pull/589 (follow-up to the Quarkus PR) > - https://github.com/apache/polaris/pull/592 (follow-up to the Quarkus PR) > - https://github.com/apache/polaris/pull/594 (fixes a sensitive issue) > > I looked briefly at all the outstanding PRs, and I think that the only one > that > might be affected significantly by the switch to Quarkus is the one that > refactors the integration tests: > > https://github.com/apache/polaris/pull/590 > > In fact, it might be a good idea to merge that one first, since it would > somewhat simplify the Quarkus PR itself. That's up for discussion, though. It > could be merged after as well. > > Also, Mike's PR that refactors PolarisAuthorizerImpl would be affected, albeit > in a lesser proportion, by the switch to Quarkus: > > https://github.com/apache/polaris/pull/465 > > I personally think that the changes in that PR are not too difficult to merge > after the switch, and I could help with that. But again, that's up for > discussion. > > Merging the Quarkus PR won't be "the end of the road" for the Quarkus switch. > There are still a few things that we need to do immediately after, like: > > - Adapting the documentation > - Adapting the Docker images > - Adapting the Helm chart > - Communicating the change to the community > > All of these are on my TODO list with high priority. > > And finally, I wanted to mention that the switch in overall aims to be as > seamless as possible for users and operators of Polaris, but a few things will > change, and we need to be prepared for that: > > - The configuration file will have a new format > - Cross-cutting concerns like logging, metrics, and tracing will have > different > shapes, names, etc. > > These are noted in the proposal document, but I wanted to mention them here as > well so that everyone is aware. > > I am hoping that the switch could be included in the next release, 0.9.0, > somewhere in January 2025. I'm looking forward to hearing your thoughts on > this. > > I wish you all a great New Year's Eve and a happy New Year! > > Best regards, > > Alex
