lasdf1234 commented on PR #12725: URL: https://github.com/apache/gravitino/pull/12725#issuecomment-5462976220
Revised approach: `CatalogManager.createBaseCatalog` already puts plaintext into catalog \`conf\` via \`secretManager.toPlaintextProperties(...)\`. The bug was that \`propertiesWithCredentialProviders()\` read raw \`entity\` props (URNs) instead of \`conf\`. Fix is only in \`propertiesWithCredentialProviders()\`: prefer \`conf\` when set, fall back to entity props. No \`SecretManager\` field / \`GravitinoEnv\` lookup on \`BaseCatalog\`. -- 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]
