ChaladiMohanVamsi commented on PR #11577: URL: https://github.com/apache/iceberg/pull/11577#issuecomment-2543179537
> @ChaladiMohanVamsi Can you help to understand how you would initialize ADLSFileIO with these changes. Priorly we can call initialize with the SAS token or account key and initialize the credentials, can u share the sample code as to how you had initialized ADLSFileIO and tried testing with these changes ? @ShyamalaGowri There is no change in the way we initialize the FileIO. The initialization remains same ([ref](https://github.com/apache/iceberg/blob/540d6a6251e31b232fe6ed2413680621454d107a/core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java#L1034)). This change is backward compatible, we can also initialize with the SAS token (existing way). This change introduces a new capability for ADLSFileIO to dynamically fetch sas token from the credential endpoint provided in the config during initialization. -- 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...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
