snvijaya commented on a change in pull request #2034:
URL: https://github.com/apache/hadoop/pull/2034#discussion_r431382991
##########
File path:
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AbfsConfiguration.java
##########
@@ -325,31 +325,91 @@ public String getPasswordString(String key) throws
IOException {
}
/**
- * Returns the account-specific Class if it exists, then looks for an
- * account-agnostic value, and finally tries the default value.
+ * Returns account-specific token provider class if it exists, else checks if
+ * an account-agnostic setting is present for token provider class if
AuthType
+ * matches with authType passed.
+ * @param authType AuthType effective on the account
* @param name Account-agnostic configuration key
* @param defaultValue Class returned if none is configured
* @param xface Interface shared by all possible values
+ * @param <U> Interface class type
* @return Highest-precedence Class object that was found
*/
- public <U> Class<? extends U> getClass(String name, Class<? extends U>
defaultValue, Class<U> xface) {
+ public <U> Class<? extends U> getTokenProviderClass(AuthType authType,
Review comment:
Inputs AuthType, name of the relevant TokenProvider config key, xface
(interface) are derived by the caller of getTokenProviderClass based on
account-specific config settings. As it applies to all the inputs equally and
is clear from the calling method's perspective, will retain the naming.
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
