bilaharith commented on a change in pull request #2034:
URL: https://github.com/apache/hadoop/pull/2034#discussion_r431000224
##########
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:
nit: I could see it from the javadocs, still naming it something like
authTypeForAccount could improve readability
----------------------------------------------------------------
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]
