eric-maynard commented on PR #1124: URL: https://github.com/apache/polaris/pull/1124#issuecomment-2704829351
> since polaris-service-common does not depend on Quarkus at all. But I'm open to revisit that as well. I see. Yeah, I am also open to revisiting that if it simplifies things. For now, what do you think is the best way to proceed? Should we introduce the flags with something like this change, and then try to refactor PolarisConfiguration etc. after? I'm a little worried about trying to sort out the refactor now (especially something potentially controversial like adding a Quarkus dependency in polaris-sevice-common) since other PRs like #1068 that should be gated behind a flag will be blocked. -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
