singhpk234 opened a new pull request, #2633: URL: https://github.com/apache/polaris/pull/2633
### About the change For the credential refresh endpoint for the ADLS its expected the expiration time should be sent with the prefix of `AZURE_SAS_TOKEN_EXPIRES_AT_MS_PREFIX` which presently Polaris doesn't sends as the course of that the CredentialProvider responsible for the refresh fails the assertion here https://github.com/apache/iceberg/blob/main/azure/src/main/java/org/apache/iceberg/azure/adlsv2/VendedAdlsCredentialProvider.java#L123 and hence not making it usable. P.S I think we should also use constants for iceberg SDK in the StorageAccessProperty, but this is an orthogonal discussion for now. ### TODO Add a small UT -- 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]
