AnanyaSingh2121 commented on code in PR #3804:
URL: https://github.com/apache/ambari/pull/3804#discussion_r1708685015


##########
ambari-server/src/main/resources/stacks/BIGTOP/3.2.0/services/HDFS/package/scripts/params_linux.py:
##########
@@ -294,6 +295,38 @@
 
 data_dir_mount_file = 
"/var/lib/ambari-agent/data/datanode/dfs_data_dir_mount.hist"
 
+router_address = None
+if 'dfs.federation.router.rpc-address' in 
config['configurations']['hdfs-rbf-site']:
+  router_rpcaddress = 
config['configurations']['hdfs-rbf-site']['dfs.federation.router.rpc-address']
+  router_address = format("hdfs://{router_rpcaddress}")
+else:
+  router_address = config['configurations']['core-site']['fs.defaultFS']
+if router_host:

Review Comment:
   adding a check for router hosts. These are configurations that will be 
loaded and populated during the Add Router wizard but only on the hosts that 
are seleted for Router installation. This will prevent any kind of 
configuration conflict due to some configs with other HDFS components due to 
ambari central config management.



-- 
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]

Reply via email to