[ https://issues.apache.org/jira/browse/HBASE-17391?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16552014#comment-16552014 ]
Hadoop QA commented on HBASE-17391: ----------------------------------- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 10s{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} master Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 4m 44s{color} | {color:green} master passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 11s{color} | {color:green} master passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 4m 45s{color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} rubocop {color} | {color:red} 0m 16s{color} | {color:red} The patch generated 2 new + 708 unchanged - 4 fixed = 710 total (was 712) {color} | | {color:orange}-0{color} | {color:orange} ruby-lint {color} | {color:orange} 0m 21s{color} | {color:orange} The patch generated 13 new + 1322 unchanged - 0 fixed = 1335 total (was 1322) {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} javadoc {color} | {color:green} 0m 11s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 6m 49s{color} | {color:green} hbase-shell in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 9s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 17m 51s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hbase:b002b0b | | JIRA Issue | HBASE-17391 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12932594/HBASE-17391.master.006.patch | | Optional Tests | asflicense javac javadoc unit rubocop ruby_lint | | uname | Linux f78f737b6d2f 3.13.0-143-generic #192-Ubuntu SMP Tue Feb 27 10:45:36 UTC 2018 x86_64 GNU/Linux | | Build tool | maven | | Personality | /home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/component/dev-support/hbase-personality.sh | | git revision | master / b4759ce6e7 | | maven | version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z) | | Default Java | 1.8.0_171 | | rubocop | v0.58.1 | | rubocop | https://builds.apache.org/job/PreCommit-HBASE-Build/13724/artifact/patchprocess/diff-patch-rubocop.txt | | ruby-lint | v2.3.1 | | ruby-lint | https://builds.apache.org/job/PreCommit-HBASE-Build/13724/artifact/patchprocess/diff-patch-ruby-lint.txt | | Test Results | https://builds.apache.org/job/PreCommit-HBASE-Build/13724/testReport/ | | Max. process+thread count | 2174 (vs. ulimit of 10000) | | modules | C: hbase-shell U: hbase-shell | | Console output | https://builds.apache.org/job/PreCommit-HBASE-Build/13724/console | | Powered by | Apache Yetus 0.7.0 http://yetus.apache.org | This message was automatically generated. > [Shell] Add shell command to get list of servers, with filters > -------------------------------------------------------------- > > Key: HBASE-17391 > URL: https://issues.apache.org/jira/browse/HBASE-17391 > Project: HBase > Issue Type: Improvement > Components: shell > Affects Versions: 1.3.0 > Reporter: Lars George > Assignee: Sreeram Venkatasubramanian > Priority: Major > Labels: beginner > Fix For: 3.0.0 > > Attachments: HBASE-17391.master.000.patch, > HBASE-17391.master.001.patch, HBASE-17391.master.002.patch, > HBASE-17391.master.003.patch, HBASE-17391.master.004.patch, > HBASE-17391.master.005.patch, HBASE-17391.master.006.patch > > > For some operations, for example calling {{update_config}}, the user needs to > specify the full server name. For region servers that is easier to find, but > not so much for the master (using {{zk_dump}} works but is noisy). It woould > be good to add a utility call that lists the servers, preferably with an > optional filter (a regexp, server type, or globbing style format) that allows > to whittle down the potentially long is of servers. For example: > {noformat} > hbase(main):001:0> list_servers "master" > master-1.internal.larsgeorge.com,16000,1483018890074 > hbase(main):002:0> list_servers "rs" > slave-1.internal.larsgeorge.com,16020,1482996572051 > slave-3.internal.larsgeorge.com,16020,1482996572481 > slave-2.internal.larsgeorge.com,16020,1482996570909 > hbase(main):003:0> list_servers "rs:s.*\.com.*" > slave-1.internal.larsgeorge.com,16020,1482996572051 > slave-3.internal.larsgeorge.com,16020,1482996572481 > slave-2.internal.larsgeorge.com,16020,1482996570909 > hbase(main):004:0> list_servers ":.*160?0.*" > master-1.internal.larsgeorge.com,16000,1483018890074 > slave-1.internal.larsgeorge.com,16020,1482996572051 > slave-3.internal.larsgeorge.com,16020,1482996572481 > slave-2.internal.larsgeorge.com,16020,1482996570909 > {noformat} > I could imagine to have {{master}}, {{backup-master}}, {{rs}}, and maybe even > {{zk}} too. The optional regexp shown uses a colon as a divider. This > combines the "by-type", and using a filter. Example #4 skips the type and > only is using the filter. > Of course, you could also implement this differently, say with two > parameters... just suggesting. -- This message was sent by Atlassian JIRA (v7.6.3#76005)