This is an automated email from the ASF dual-hosted git repository. ayushsaxena pushed a commit to branch trunk in repository https://gitbox.apache.org/repos/asf/hadoop.git
The following commit(s) were added to refs/heads/trunk by this push: new b06c234 HDFS-14713. RBF: RouterAdmin supports refreshRouterArgs command but not on display. Contributed by wangzhaohui. b06c234 is described below commit b06c2345efffde2955b8c2d5fd954ad73b5d8677 Author: Ayush Saxena <ayushsax...@apache.org> AuthorDate: Wed Aug 14 20:22:24 2019 +0530 HDFS-14713. RBF: RouterAdmin supports refreshRouterArgs command but not on display. Contributed by wangzhaohui. --- .../org/apache/hadoop/hdfs/tools/federation/RouterAdmin.java | 2 +- .../hdfs/server/federation/router/TestRouterAdminCLI.java | 10 +++++++++- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/tools/federation/RouterAdmin.java b/hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/tools/federation/RouterAdmin.java index 4b50534..b39ed96 100644 --- a/hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/tools/federation/RouterAdmin.java +++ b/hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/tools/federation/RouterAdmin.java @@ -127,7 +127,7 @@ public class RouterAdmin extends Configured implements Tool { {"-add", "-update", "-rm", "-ls", "-getDestination", "-setQuota", "-clrQuota", "-safemode", "-nameservice", "-getDisabledNameservices", - "-refresh"}; + "-refresh", "-refreshRouterArgs"}; StringBuilder usage = new StringBuilder(); usage.append("Usage: hdfs dfsrouteradmin :\n"); for (int i = 0; i < commands.length; i++) { diff --git a/hadoop-hdfs-project/hadoop-hdfs-rbf/src/test/java/org/apache/hadoop/hdfs/server/federation/router/TestRouterAdminCLI.java b/hadoop-hdfs-project/hadoop-hdfs-rbf/src/test/java/org/apache/hadoop/hdfs/server/federation/router/TestRouterAdminCLI.java index 5a20a70..731d5d4 100644 --- a/hadoop-hdfs-project/hadoop-hdfs-rbf/src/test/java/org/apache/hadoop/hdfs/server/federation/router/TestRouterAdminCLI.java +++ b/hadoop-hdfs-project/hadoop-hdfs-rbf/src/test/java/org/apache/hadoop/hdfs/server/federation/router/TestRouterAdminCLI.java @@ -576,6 +576,12 @@ public class TestRouterAdminCLI { assertTrue(out.toString().contains("\t[-getDestination <path>]")); out.reset(); + argv = new String[] {"-refreshRouterArgs"}; + assertEquals(-1, ToolRunner.run(admin, argv)); + assertTrue(out.toString().contains("\t[-refreshRouterArgs " + + "<host:ipc_port> <key> [arg1..argn]]")); + out.reset(); + argv = new String[] {"-Random"}; assertEquals(-1, ToolRunner.run(admin, argv)); String expected = "Usage: hdfs dfsrouteradmin :\n" @@ -594,7 +600,9 @@ public class TestRouterAdminCLI { + "<quota in bytes or quota size string>]\n" + "\t[-clrQuota <path>]\n" + "\t[-safemode enter | leave | get]\n" + "\t[-nameservice enable | disable <nameservice>]\n" - + "\t[-getDisabledNameservices]"; + + "\t[-getDisabledNameservices]\n" + + "\t[-refresh]\n" + + "\t[-refreshRouterArgs <host:ipc_port> <key> [arg1..argn]]"; assertTrue("Wrong message: " + out, out.toString().contains(expected)); out.reset(); } --------------------------------------------------------------------- To unsubscribe, e-mail: common-commits-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-commits-h...@hadoop.apache.org