snazy opened a new pull request, #2277: URL: https://github.com/apache/polaris/pull/2277
The `persistStorageIntegrationIfNeeded` function does nothing in any implementation, therefore the corresponding load-function cannot yield anything ever. This change removes both functions. That allows removing the `createStorageIntegration` function as well and let its only call site `getSubscopedCredsForEntity` be implemented once and directly call the storage integration provider. Being able to access object storage without access to entities is a requirement for all currently active tasks proposals. This is one step towards that goal. A follow-up will entirely remove the storage-integration-provider dependency from persistence and let `StorageCredentialCache` deal with it as the only call site. -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org