[jira] [Commented] (HDFS-14713) RBF: routeradmin support refreshRouterArgs command but it not on display
[ https://issues.apache.org/jira/browse/HDFS-14713?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16906361#comment-16906361 ] Ayush Saxena commented on HDFS-14713: - Thanx [~wangzhaohui] v003 LGTM +1 > RBF: routeradmin support refreshRouterArgs command but it not on display > > > Key: HDFS-14713 > URL: https://issues.apache.org/jira/browse/HDFS-14713 > Project: Hadoop HDFS > Issue Type: Sub-task >Reporter: wangzhaohui >Assignee: wangzhaohui >Priority: Major > Attachments: HDFS-14713-000.patch, HDFS-14713-001.patch, > HDFS-14713-002.patch, HDFS-14713-003.patch, after.png, before.png > > > When the cmd commond is null,the refreshRouterArgs command is not display > ,because there is one missing value in the String[] commands > {code:java} > // > if (cmd == null) { > String[] commands = > {"-add", "-update", "-rm", "-ls", "-getDestination", > "-setQuota", "-clrQuota", > "-safemode", "-nameservice", "-getDisabledNameservices", > "-refresh"}; > StringBuilder usage = new StringBuilder(); > usage.append("Usage: hdfs dfsrouteradmin :\n"); > for (int i = 0; i < commands.length; i++) { > usage.append(getUsage(commands[i])); > if (i + 1 < commands.length) { > usage.append("\n"); > } > } > > } > {code} -- This message was sent by Atlassian JIRA (v7.6.14#76016) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-14713) RBF: routeradmin support refreshRouterArgs command but it not on display
[ https://issues.apache.org/jira/browse/HDFS-14713?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16905884#comment-16905884 ] wangzhaohui commented on HDFS-14713: Hi,[~ayushtkn] [~elgoiri] upload v003,please review,thanks. > RBF: routeradmin support refreshRouterArgs command but it not on display > > > Key: HDFS-14713 > URL: https://issues.apache.org/jira/browse/HDFS-14713 > Project: Hadoop HDFS > Issue Type: Sub-task >Reporter: wangzhaohui >Assignee: wangzhaohui >Priority: Major > Attachments: HDFS-14713-000.patch, HDFS-14713-001.patch, > HDFS-14713-002.patch, HDFS-14713-003.patch, after.png, before.png > > > When the cmd commond is null,the refreshRouterArgs command is not display > ,because there is one missing value in the String[] commands > {code:java} > // > if (cmd == null) { > String[] commands = > {"-add", "-update", "-rm", "-ls", "-getDestination", > "-setQuota", "-clrQuota", > "-safemode", "-nameservice", "-getDisabledNameservices", > "-refresh"}; > StringBuilder usage = new StringBuilder(); > usage.append("Usage: hdfs dfsrouteradmin :\n"); > for (int i = 0; i < commands.length; i++) { > usage.append(getUsage(commands[i])); > if (i + 1 < commands.length) { > usage.append("\n"); > } > } > > } > {code} -- This message was sent by Atlassian JIRA (v7.6.14#76016) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-14713) RBF: routeradmin support refreshRouterArgs command but it not on display
[ https://issues.apache.org/jira/browse/HDFS-14713?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16905773#comment-16905773 ] Ayush Saxena commented on HDFS-14713: - Thanx [~wangzhaohui] for the patch. Add the part here in the test too : {code:java} argv = new String[] {"-Random"}; assertEquals(-1, ToolRunner.run(admin, argv)); String expected = "Usage: hdfs dfsrouteradmin :\n" + "\t[-add" + "[-readonly] [-faulttolerant] " + "[-order HASH|LOCAL|RANDOM|HASH_ALL|SPACE] " + "-owner -group -mode ]\n" + "\t[-update [ " + "] [-readonly true|false]" + " [-faulttolerant true|false] " + "[-order HASH|LOCAL|RANDOM|HASH_ALL|SPACE] " + "-owner -group -mode ]\n" + "\t[-rm ]\n" + "\t[-ls ]\n" + "\t[-getDestination ]\n" + "\t[-setQuota -nsQuota -ssQuota " + "]\n" + "\t[-clrQuota ]\n" + "\t[-safemode enter | leave | get]\n" + "\t[-nameservice enable | disable ]\n" + "\t[-getDisabledNameservices]"; {code} > RBF: routeradmin support refreshRouterArgs command but it not on display > > > Key: HDFS-14713 > URL: https://issues.apache.org/jira/browse/HDFS-14713 > Project: Hadoop HDFS > Issue Type: Sub-task >Reporter: wangzhaohui >Assignee: wangzhaohui >Priority: Major > Attachments: HDFS-14713-000.patch, HDFS-14713-001.patch, > HDFS-14713-002.patch, after.png, before.png > > > When the cmd commond is null,the refreshRouterArgs command is not display > ,because there is one missing value in the String[] commands > {code:java} > // > if (cmd == null) { > String[] commands = > {"-add", "-update", "-rm", "-ls", "-getDestination", > "-setQuota", "-clrQuota", > "-safemode", "-nameservice", "-getDisabledNameservices", > "-refresh"}; > StringBuilder usage = new StringBuilder(); > usage.append("Usage: hdfs dfsrouteradmin :\n"); > for (int i = 0; i < commands.length; i++) { > usage.append(getUsage(commands[i])); > if (i + 1 < commands.length) { > usage.append("\n"); > } > } > > } > {code} -- This message was sent by Atlassian JIRA (v7.6.14#76016) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-14713) RBF: routeradmin support refreshRouterArgs command but it not on display
[ https://issues.apache.org/jira/browse/HDFS-14713?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16905771#comment-16905771 ] Hadoop QA commented on HDFS-14713: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 49s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 1 new or modified test files. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 19m 52s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 32s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 21s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 36s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 12m 46s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 0m 59s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 37s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 32s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 26s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 26s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 17s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 33s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 13m 1s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 1s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 33s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:red}-1{color} | {color:red} unit {color} | {color:red} 23m 50s{color} | {color:red} hadoop-hdfs-rbf in the patch failed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 27s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 78m 5s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | Failed junit tests | hadoop.hdfs.server.federation.router.TestRouterWithSecureStartup | | | hadoop.hdfs.server.federation.security.TestRouterHttpDelegationToken | \\ \\ || Subsystem || Report/Notes || | Docker | Client=18.09.7 Server=18.09.7 Image:yetus/hadoop:bdbca0e53b4 | | JIRA Issue | HDFS-14713 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12977428/HDFS-14713-002.patch | | Optional Tests | dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux 79d8280416ab 4.15.0-52-generic #56-Ubuntu SMP Tue Jun 4 22:49:08 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / c92b498 | | maven | version: Apache Maven 3.3.9 | | Default Java | 1.8.0_212 | | findbugs | v3.1.0-RC1 | | unit | https://builds.apache.org/job/PreCommit-HDFS-Build/27488/artifact/out/patch-unit-hadoop-hdfs-project_hadoop-hdfs-rbf.txt | | Test Results | https://builds.apache.org/job/PreCommit-HDFS-Build/27488/testReport/ | | Max. process+thread count | 1584 (vs. ulimit of 5500) | | modules | C: hadoop-hdfs-project/hadoop-hdfs-rbf U: hadoop-hdfs-project/hadoop-hdfs-rbf | | Console output |
[jira] [Commented] (HDFS-14713) RBF: routeradmin support refreshRouterArgs command but it not on display
[ https://issues.apache.org/jira/browse/HDFS-14713?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16905352#comment-16905352 ] Íñigo Goiri commented on HDFS-14713: [^HDFS-14713-001.patch] LGTM. +1 > RBF: routeradmin support refreshRouterArgs command but it not on display > > > Key: HDFS-14713 > URL: https://issues.apache.org/jira/browse/HDFS-14713 > Project: Hadoop HDFS > Issue Type: Sub-task >Reporter: wangzhaohui >Assignee: wangzhaohui >Priority: Major > Attachments: HDFS-14713-000.patch, HDFS-14713-001.patch, after.png, > before.png > > > When the cmd commond is null,the refreshRouterArgs command is not display > ,because there is one missing value in the String[] commands > {code:java} > // > if (cmd == null) { > String[] commands = > {"-add", "-update", "-rm", "-ls", "-getDestination", > "-setQuota", "-clrQuota", > "-safemode", "-nameservice", "-getDisabledNameservices", > "-refresh"}; > StringBuilder usage = new StringBuilder(); > usage.append("Usage: hdfs dfsrouteradmin :\n"); > for (int i = 0; i < commands.length; i++) { > usage.append(getUsage(commands[i])); > if (i + 1 < commands.length) { > usage.append("\n"); > } > } > > } > {code} -- This message was sent by Atlassian JIRA (v7.6.14#76016) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-14713) RBF: routeradmin support refreshRouterArgs command but it not on display
[ https://issues.apache.org/jira/browse/HDFS-14713?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16904600#comment-16904600 ] Ayush Saxena commented on HDFS-14713: - Completely agree, this should be covered. [~wangzhaohui] would be great if you can add one test at {{TestRouterAdminCli}} to assert the full usage message. > RBF: routeradmin support refreshRouterArgs command but it not on display > > > Key: HDFS-14713 > URL: https://issues.apache.org/jira/browse/HDFS-14713 > Project: Hadoop HDFS > Issue Type: Sub-task >Reporter: wangzhaohui >Assignee: wangzhaohui >Priority: Major > Attachments: HDFS-14713-000.patch, after.png, before.png > > > When the cmd commond is null,the refreshRouterArgs command is not display > ,because there is one missing value in the String[] commands > {code:java} > // > if (cmd == null) { > String[] commands = > {"-add", "-update", "-rm", "-ls", "-getDestination", > "-setQuota", "-clrQuota", > "-safemode", "-nameservice", "-getDisabledNameservices", > "-refresh"}; > StringBuilder usage = new StringBuilder(); > usage.append("Usage: hdfs dfsrouteradmin :\n"); > for (int i = 0; i < commands.length; i++) { > usage.append(getUsage(commands[i])); > if (i + 1 < commands.length) { > usage.append("\n"); > } > } > > } > {code} -- This message was sent by Atlassian JIRA (v7.6.14#76016) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-14713) RBF: routeradmin support refreshRouterArgs command but it not on display
[ https://issues.apache.org/jira/browse/HDFS-14713?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16904189#comment-16904189 ] Íñigo Goiri commented on HDFS-14713: [~ayushtkn], I think we should actually cover it :) [~wangzhaohui] do you mind extending the test too? > RBF: routeradmin support refreshRouterArgs command but it not on display > > > Key: HDFS-14713 > URL: https://issues.apache.org/jira/browse/HDFS-14713 > Project: Hadoop HDFS > Issue Type: Sub-task >Reporter: wangzhaohui >Assignee: wangzhaohui >Priority: Major > Attachments: HDFS-14713-000.patch, after.png, before.png > > > When the cmd commond is null,the refreshRouterArgs command is not display > ,because there is one missing value in the String[] commands > {code:java} > // > if (cmd == null) { > String[] commands = > {"-add", "-update", "-rm", "-ls", "-getDestination", > "-setQuota", "-clrQuota", > "-safemode", "-nameservice", "-getDisabledNameservices", > "-refresh"}; > StringBuilder usage = new StringBuilder(); > usage.append("Usage: hdfs dfsrouteradmin :\n"); > for (int i = 0; i < commands.length; i++) { > usage.append(getUsage(commands[i])); > if (i + 1 < commands.length) { > usage.append("\n"); > } > } > > } > {code} -- This message was sent by Atlassian JIRA (v7.6.14#76016) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-14713) RBF: routeradmin support refreshRouterArgs command but it not on display
[ https://issues.apache.org/jira/browse/HDFS-14713?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16904154#comment-16904154 ] Ayush Saxena commented on HDFS-14713: - Ohh, my bad.. I thought the output would be covered as part of test. Anyway the fix is direct. LGTM > RBF: routeradmin support refreshRouterArgs command but it not on display > > > Key: HDFS-14713 > URL: https://issues.apache.org/jira/browse/HDFS-14713 > Project: Hadoop HDFS > Issue Type: Sub-task >Reporter: wangzhaohui >Assignee: wangzhaohui >Priority: Major > Attachments: HDFS-14713-000.patch, after.png, before.png > > > When the cmd commond is null,the refreshRouterArgs command is not display > ,because there is one missing value in the String[] commands > {code:java} > // > if (cmd == null) { > String[] commands = > {"-add", "-update", "-rm", "-ls", "-getDestination", > "-setQuota", "-clrQuota", > "-safemode", "-nameservice", "-getDisabledNameservices", > "-refresh"}; > StringBuilder usage = new StringBuilder(); > usage.append("Usage: hdfs dfsrouteradmin :\n"); > for (int i = 0; i < commands.length; i++) { > usage.append(getUsage(commands[i])); > if (i + 1 < commands.length) { > usage.append("\n"); > } > } > > } > {code} -- This message was sent by Atlassian JIRA (v7.6.14#76016) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-14713) RBF: routeradmin support refreshRouterArgs command but it not on display
[ https://issues.apache.org/jira/browse/HDFS-14713?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16904134#comment-16904134 ] Ayush Saxena commented on HDFS-14713: - This if IIRC would require change in TestRouterAdminCli too > RBF: routeradmin support refreshRouterArgs command but it not on display > > > Key: HDFS-14713 > URL: https://issues.apache.org/jira/browse/HDFS-14713 > Project: Hadoop HDFS > Issue Type: Sub-task >Reporter: wangzhaohui >Assignee: wangzhaohui >Priority: Major > Attachments: HDFS-14713-000.patch, after.png, before.png > > > When the cmd commond is null,the refreshRouterArgs command is not display > ,because there is one missing value in the String[] commands > {code:java} > // > if (cmd == null) { > String[] commands = > {"-add", "-update", "-rm", "-ls", "-getDestination", > "-setQuota", "-clrQuota", > "-safemode", "-nameservice", "-getDisabledNameservices", > "-refresh"}; > StringBuilder usage = new StringBuilder(); > usage.append("Usage: hdfs dfsrouteradmin :\n"); > for (int i = 0; i < commands.length; i++) { > usage.append(getUsage(commands[i])); > if (i + 1 < commands.length) { > usage.append("\n"); > } > } > > } > {code} -- This message was sent by Atlassian JIRA (v7.6.14#76016) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-14713) RBF: routeradmin support refreshRouterArgs command but it not on display
[ https://issues.apache.org/jira/browse/HDFS-14713?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16904135#comment-16904135 ] Hadoop QA commented on HDFS-14713: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 37s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:red}-1{color} | {color:red} test4tests {color} | {color:red} 0m 0s{color} | {color:red} The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 17m 49s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 25s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 18s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 28s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 10m 42s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 0m 53s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 30s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 28s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 21s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 21s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 16s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 34s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 12m 8s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 0m 53s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 31s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:red}-1{color} | {color:red} unit {color} | {color:red} 21m 50s{color} | {color:red} hadoop-hdfs-rbf in the patch failed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 24s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 69m 51s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | Failed junit tests | hadoop.hdfs.server.federation.security.TestRouterHttpDelegationToken | | | hadoop.hdfs.server.federation.router.TestRouterWithSecureStartup | \\ \\ || Subsystem || Report/Notes || | Docker | Client=19.03.1 Server=19.03.1 Image:yetus/hadoop:bdbca0e | | JIRA Issue | HDFS-14713 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12977140/HDFS-14713-000.patch | | Optional Tests | dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux 2d61dcf798f7 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / a79564f | | maven | version: Apache Maven 3.3.9 | | Default Java | 1.8.0_222 | | findbugs | v3.1.0-RC1 | | unit | https://builds.apache.org/job/PreCommit-HDFS-Build/27456/artifact/out/patch-unit-hadoop-hdfs-project_hadoop-hdfs-rbf.txt | | Test Results | https://builds.apache.org/job/PreCommit-HDFS-Build/27456/testReport/ | | Max. process+thread count | 1616 (vs. ulimit of 1) | | modules | C:
[jira] [Commented] (HDFS-14713) RBF: routeradmin support refreshRouterArgs command but it not on display
[ https://issues.apache.org/jira/browse/HDFS-14713?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16904087#comment-16904087 ] Íñigo Goiri commented on HDFS-14713: This was missed in HDFS-13856. > RBF: routeradmin support refreshRouterArgs command but it not on display > > > Key: HDFS-14713 > URL: https://issues.apache.org/jira/browse/HDFS-14713 > Project: Hadoop HDFS > Issue Type: Sub-task >Reporter: wangzhaohui >Assignee: wangzhaohui >Priority: Major > Attachments: HDFS-14713-000.patch, after.png, before.png > > > When the cmd commond is null,the refreshRouterArgs command is not display > ,because there is one missing value in the String[] commands > {code:java} > // > if (cmd == null) { > String[] commands = > {"-add", "-update", "-rm", "-ls", "-getDestination", > "-setQuota", "-clrQuota", > "-safemode", "-nameservice", "-getDisabledNameservices", > "-refresh"}; > StringBuilder usage = new StringBuilder(); > usage.append("Usage: hdfs dfsrouteradmin :\n"); > for (int i = 0; i < commands.length; i++) { > usage.append(getUsage(commands[i])); > if (i + 1 < commands.length) { > usage.append("\n"); > } > } > > } > {code} -- This message was sent by Atlassian JIRA (v7.6.14#76016) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org