adnanhemani commented on issue #3196: URL: https://github.com/apache/polaris/issues/3196#issuecomment-3615595454
@tokoko I took a look at this today in depth. I don't see any other way to propagate the Principal information into `AwsCredentialsStorageIntegration` other than passing it in through the call chain - likely starting at `StorageAccessConfigProvider.java` where we can inject in the `PolarisPrincipal` bean. I looked into some other ways of injecting the principal information directly into `AwsCredentialsStorageIntegration` but I don't exactly see a workable path :( If 3170 stays in its current form, we will have to plumb the principal information through separately. If you'd like to make this more thinly-scoped change, I'm very glad to help support you in making that change. Or if you'd prefer someone else pick it up, I may have time later. Let me know your thoughts and if this is a high-priority change you need - we can figure out how to make this more isolated change sooner than later. -- 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]
