[
https://issues.apache.org/jira/browse/HDFS-17213?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17771035#comment-17771035
]
ASF GitHub Bot commented on HDFS-17213:
---------------------------------------
ayushtkn commented on code in PR #6134:
URL: https://github.com/apache/hadoop/pull/6134#discussion_r1342504971
##########
hadoop-hdfs-project/hadoop-hdfs-rbf/src/test/java/org/apache/hadoop/hdfs/server/federation/router/TestRouterAdminCLI.java:
##########
@@ -1925,6 +1925,42 @@ public void testAddMultipleMountPointsFailure() throws
Exception {
err.toString().contains("Cannot add mount points:
[/testAddMultiMountPoints-01"));
}
+ @Test
+ public void testGetUsage() throws Exception {
Review Comment:
this test is pretty irrelevant in the context, you are checking the exit
codes here, where you didn't even touch them, there should be an existing test
I believe which tests the help output
> In RouterAdmin class,"if else" is not as elegant as "switch case".
> ------------------------------------------------------------------
>
> Key: HDFS-17213
> URL: https://issues.apache.org/jira/browse/HDFS-17213
> Project: Hadoop HDFS
> Issue Type: Improvement
> Components: rbf
> Affects Versions: 3.3.4
> Reporter: xiaojunxiang
> Priority: Minor
> Labels: pull-request-available
> Attachments: image-2023-10-01-21-29-22-983.png
>
>
> In RouterAdmin class, "if else" is not as elegant as "switch case" in the
> admin command parameters section.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
