You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by ay...@apache.org on 2019/08/14 15:15:20 UTC
[hadoop] branch trunk updated: HDFS-14713. RBF: RouterAdmin
supports refreshRouterArgs command but not on display. Contributed by
wangzhaohui.
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 <ay...@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-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org