adutra commented on PR #1511: URL: https://github.com/apache/polaris/pull/1511#issuecomment-2848617373
> @adutra good point on (2) -- I would be okay changing it back to remove `overrides`... to be honest, I still find the post-Quarkus method of specifying a config's value really arcane. We went from e.g. `featureConfiguration.SUPPORTED_CATALOG_STORAGE_TYPES` to `polaris.features.defaults."SUPPORTED_CATALOG_STORAGE_TYPES"`. Why is it not just `featureConfiguration.X = Y`? I'm all in to try and reduce the verbosity of the configuration options. But, I think it's also nice to always prefix our own options with "polaris." What I think we should try is to remove the "defaults." part. It should be possible imho by annotating the corresponding method with `@WithParentName`. -- 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]
