[
https://issues.apache.org/jira/browse/HDFS-14351?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16790332#comment-16790332
]
He Xiaoqiao commented on HDFS-14351:
------------------------------------
Thanks [~elgoiri] for reviewing.
[^HDFS-14351-HDFS-13891.001.patch] rebased branch HDFS-13891 and just change
configuration settings tiny about #TestRouterNamenodeMonitoring.
Do we need another more unit test for this configuration item since it has
verified nsid and nnid in #TestRouterNamenodeMonitoring?
> RBF: Optimize configuration item resolving for monitor namenode
> ---------------------------------------------------------------
>
> Key: HDFS-14351
> URL: https://issues.apache.org/jira/browse/HDFS-14351
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: rbf
> Reporter: He Xiaoqiao
> Assignee: He Xiaoqiao
> Priority: Major
> Attachments: HDFS-14351-HDFS-13891.001.patch, HDFS-14351.001.patch,
> HDFS-14351.002.patch
>
>
> We invoke {{configuration.get}} to resolve configuration item
> `dfs.federation.router.monitor.namenode` at `Router.java`, then split the
> value by comma to get nsid and nnid, it may confused users since this is not
> compatible with blank space but other common parameters could do. The
> following segment show example that resolve fails.
> {code:java}
> <property>
> <name>dfs.federation.router.monitor.namenode</name>
> <value>nameservice1.nn1, nameservice1.nn2</value>
> <description>
> The identifier of the namenodes to monitor and heartbeat.
> </description>
> </property>
> {code}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
