collado-mike commented on PR #417: URL: https://github.com/apache/polaris/pull/417#issuecomment-2461009762
> If it's generally a good idea, do you mind to hold this for a few days ? The reason that it would break the current Quarkus branch. I'm proposing: > > 1. To open Quarkus draft PR to drive discussion/consensus > 2. Review this PR after 1. > Thoughts ? Hey @jbonofre - I have a branch at https://github.com/jbonofre/polaris/compare/QUARKUS...collado-mike:polaris:patch-1?expand=1 that has this change merged into your Quarkus branch. The code compiles, though when I run the tests locally, it fails due to missing `AWS_REGION` branch. Can we merge that change into your branch and move forward with this refactoring in main? -- 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]
