roeap commented on code in PR #3581:
URL: https://github.com/apache/arrow-rs/pull/3581#discussion_r1084469125
##########
object_store/src/azure/mod.rs:
##########
@@ -824,18 +947,40 @@ impl MicrosoftAzureBuilder {
Ok(credential::CredentialProvider::AccessKey(bearer_token))
} else if let Some(access_key) = self.access_key {
Ok(credential::CredentialProvider::AccessKey(access_key))
+ } else if self.use_managed_identity {
Review Comment:
The default azure credential does a lot of opinionated things that I think
we do not want to replicate here :). That said, it makes sense to use the
metadata as default since it may work without any configuration.
--
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]
