dimas-b commented on PR #1032: URL: https://github.com/apache/polaris/pull/1032#issuecomment-2698683635
From my POV, I'd prefer educating users on how to benefit from the multitude of ways to configure Polaris (which come from the Quarkus infra). I believe it will be an advantage to all users in the long run, even though there is a learning curve. I personally think that combining multiple config sources at build time will complicate the build unnecessarily. This looks similar to users (currently) having to provide extra build parameters for PostgreSQL, which makes every build different at the binary level. Ideally, I think no rebuild should be necessary to switch between database backends or for any other runtime config changes (only a restart with different config values should do it). A few properties that do require re-compilation are very low-level and changing them is something that normally needs a PR, IMHO :) -- 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]
