poojanilangekar commented on PR #2301: URL: https://github.com/apache/polaris/pull/2301#issuecomment-3169551370
The conditional runtime dependency could rely on one of the two options: 1. Based on the `NonRESTCatalogs` property (which an additional property to be set in `gradle.properties` or as a JVM argument). 1. Reuse the existing `polaris.features.SUPPORTED_EXTERNAL_CATALOG_AUTHENTICATION_TYPES` but require the user to declare it in `gradle.properties` instead of `application.properties`. The current implementation uses option 1. I can change the option or name if necessary. If you have other possible suggestions, please let me know. -- 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]
