steveloughran commented on a change in pull request #1842: HADOOP-16730 : ABFS:
Add Authorizer Interface
URL: https://github.com/apache/hadoop/pull/1842#discussion_r382648840
##########
File path:
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AzureBlobFileSystemStore.java
##########
@@ -1130,8 +1131,9 @@ private void initializeClient(URI uri, String
fileSystemName, String accountName
SharedKeyCredentials creds = null;
AccessTokenProvider tokenProvider = null;
+ SASTokenProvider sasTokenProvider = null;
- if (abfsConfiguration.getAuthType(accountName) == AuthType.SharedKey) {
+ if (this.authType == AuthType.SharedKey) {
Review comment:
we don't normally use this. for field refs on the basis that IDEs mark
things differently anyway. But if you wan to be consistent with the existin g
code, stay with it. Though its actually a bit mixed (abfsConfiguration is not
qualified).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
