fabio-rizzo-01 commented on PR #2650: URL: https://github.com/apache/polaris/pull/2650#issuecomment-3339607371
> Thanks, @fabio-rizzo-01, for addressing the comment. I'm good with a follow-up doc PR. In that case, I suggest we remove the configuration changes from both `application.properties` files. We add a new jdbc connection option in this PR. We got two options now. Users might confused by which option to choose, how they set it, especially since neither file includes a complete setup for both. I can't remove all the properties these 2 need to be there: `quarkus.rds.sync-client.type=apache quarkus.devservices.enabled=false` the app won't run, because it needs to know which client to use to connect otherwise it defaults to the url-connection-client that is not there and for the second one it will try to start up localstack. So I can remove the db config but not these 2. -- 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]
