gh-yzou commented on code in PR #1511: URL: https://github.com/apache/polaris/pull/1511#discussion_r2085080072
########## quarkus/defaults/src/main/resources/application.properties: ########## @@ -113,8 +113,8 @@ polaris.features.defaults."SUPPORTED_CATALOG_STORAGE_TYPES"=["S3","GCS","AZURE", # polaris.features.defaults."ENABLE_CATALOG_FEDERATION"=true # realm overrides -# polaris.features.realm-overrides."my-realm"."INITIALIZE_DEFAULT_CATALOG_FILEIO_FOR_TEST"=true -# polaris.features.realm-overrides."my-realm"."SKIP_CREDENTIAL_SUBSCOPING_INDIRECTION"=true +# polaris.features.realm-overrides."my-realm".overrides."INITIALIZE_DEFAULT_CATALOG_FILEIO_FOR_TEST"=true Review Comment: @adutra Based on the test I have in https://github.com/apache/polaris/pull/1505/files, the key seems still need to be "realm-overrides" instead of "overrides". I tried couple of things with the test, 1) without reverting the change, when i use either "realm-overrides" or "overrides" , quarkus start fails with "xxx in SysPropConfigSource does not map to any root" 2) with reverting the change, "realm-overrides" works fine, but "overrides" still fails with the same error. -- 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]
