manika137 commented on code in PR #7461:
URL: https://github.com/apache/hadoop/pull/7461#discussion_r1986572444
##########
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AbfsConfiguration.java:
##########
@@ -493,7 +497,7 @@ public AbfsConfiguration(final Configuration rawConfig,
*/
public AbfsConfiguration(final Configuration rawConfig, String accountName)
Review Comment:
Taken
##########
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AbfsConfiguration.java:
##########
@@ -644,16 +658,20 @@ public int getInt(String key, int defaultValue) {
}
/**
- * Returns the account-specific password in string form if it exists, then
+ * Returns the container-specific password if it exists,
+ * else searches for the account-specific password, else finally
* looks for an account-agnostic value.
* @param key Account-agnostic configuration key
* @return value in String form if one exists, else null
* @throws IOException if parsing fails.
*/
public String getPasswordString(String key) throws IOException {
- char[] passchars = rawConfig.getPassword(accountConf(key));
- if (passchars == null) {
- passchars = rawConfig.getPassword(key);
+ char[] passchars = rawConfig.getPassword(containerConf(key));
+ if(passchars == null) {
+ passchars = rawConfig.getPassword(accountConf(key));
+ if(passchars == null){
+ passchars = rawConfig.getPassword(key);
+ }
Review Comment:
Taken
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
