XN137 opened a new pull request, #2133:
URL: https://github.com/apache/polaris/pull/2133

   `PolarisEntityManager` itself is not using the `StorageCredentialCache` but 
just hands it out via `getCredentialCache`.
   the only caller of `getCredentialCache` is `FileIOUtil.refreshAccessConfig`, 
which in in turn is only called by `DefaultFileIOFactory` and `IcebergCatalog`.
   
   note that in a follow-up we will likely be able to remove 
`PolarisEntityManager` usage completely from `IcebergCatalog`.
   
   additional cleanups:
   - use `StorageCredentialCache` injection in tests (but we need to invalidate 
all entries on test start)
   - remove unused `UserSecretsManagerFactory` from 
`PolarisCallContextCatalogFactory`


-- 
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]

Reply via email to