felipecrv commented on code in PR #39321: URL: https://github.com/apache/arrow/pull/39321#discussion_r1435273720
########## cpp/src/arrow/filesystem/azurefs.cc: ########## @@ -119,6 +119,14 @@ Status AzureOptions::ConfigureDefaultCredential(const std::string& account_name) return Status::OK(); } +Status AzureOptions::ConfigureManagedIdentityCredential(const std::string& account_name, Review Comment: This would be a good idea in normal classes, but these `*Options` classes tend to be designed like simple C structs that can be constructed from any language and easily serialized. The only exception being the secret stuff that we don't want to leak and lazily initialized fields. -- 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]
