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

Reply via email to