flyrain commented on PR #3341: URL: https://github.com/apache/polaris/pull/3341#issuecomment-3711514719
Thanks for the PR, @dimas-b ! Would it be possible to move application.properties from runtime/defaults to runtime/server instead? That would align with the approach in the admin module, where the application.properties lives with the runnable modules. I think this is to consolidate config files from both server and default modules to avoid conflicts. Question on potential user side conflicts, what happens if users add their own application.properties in the binary distribution or Docker image? Would it conflict with the one coming from runtime/defaults, and if so, which one wins? -- 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]
