AnanyaSingh2121 commented on code in PR #6982:
URL: https://github.com/apache/hadoop/pull/6982#discussion_r1720770936
##########
hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/tools/federation/RouterAdmin.java:
##########
@@ -1364,7 +1364,12 @@ public int genericRefresh(String[] argv, int i) throws
IOException {
* @throws IOException if the operation was not successful.
*/
private int refreshCallQueue() throws IOException {
+ // for security authorization
+ // server principal for this call
+ // should be Routers's one.
Configuration conf = getConf();
+ conf.set(CommonConfigurationKeys.HADOOP_SECURITY_SERVICE_USER_NAME_KEY,
Review Comment:
It is set when the refreshcallqueue is executed. For reference in DFSAdmin
we handle it this way
https://github.com/apache/hadoop/blob/b5f88990b721aebe84d4bbf97ed2fdfbd0f42854/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/tools/DFSAdmin.java#L1875
Also the key is defined in
https://github.com/apache/hadoop/blob/b5f88990b721aebe84d4bbf97ed2fdfbd0f42854/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java#L668
--
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]
