[GitHub] [hbase] Apache-HBase commented on issue #722: HBASE-23065 [hbtop] Top-N heavy hitter user and client drill downs

2019-11-18 Thread GitBox
Apache-HBase commented on issue #722: HBASE-23065 [hbtop] Top-N heavy hitter 
user and client drill downs
URL: https://github.com/apache/hbase/pull/722#issuecomment-554918864
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   0m 32s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  0s |  No case conflicting files 
found.  |
   | +0 :ok: |  prototool  |   0m  1s |  prototool was not available.  |
   | +1 :green_heart: |  hbaseanti  |   0m  0s |  Patch does not have any 
anti-patterns.  |
   | +1 :green_heart: |  @author  |   0m  0s |  The patch does not contain any 
@author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  The patch appears to include 
7 new or modified test files.  |
   ||| _ master Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 34s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   5m 19s |  master passed  |
   | +1 :green_heart: |  compile  |   3m 23s |  master passed  |
   | +1 :green_heart: |  checkstyle  |   3m  1s |  master passed  |
   | +1 :green_heart: |  shadedjars  |   4m 33s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   2m 18s |  master passed  |
   | +0 :ok: |  spotbugs  |   4m  2s |  Used deprecated FindBugs config; 
considering switching to SpotBugs.  |
   | +1 :green_heart: |  findbugs  |  10m 25s |  master passed  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 15s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   4m 56s |  the patch passed  |
   | +1 :green_heart: |  compile  |   3m 20s |  the patch passed  |
   | +1 :green_heart: |  cc  |   3m 20s |  the patch passed  |
   | +1 :green_heart: |  javac  |   3m 20s |  the patch passed  |
   | +1 :green_heart: |  checkstyle  |   2m 59s |  the patch passed  |
   | +1 :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | +1 :green_heart: |  shadedjars  |   4m 34s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  hadoopcheck  |  15m 39s |  Patch does not cause any 
errors with Hadoop 2.8.5 2.9.2 or 3.1.2.  |
   | +1 :green_heart: |  hbaseprotoc  |   3m 21s |  the patch passed  |
   | +1 :green_heart: |  javadoc  |   2m 17s |  the patch passed  |
   | +1 :green_heart: |  findbugs  |  11m 26s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  |   0m 42s |  hbase-protocol-shaded in the patch 
passed.  |
   | +1 :green_heart: |  unit  |   0m 32s |  hbase-hadoop-compat in the patch 
passed.  |
   | +1 :green_heart: |  unit  |   0m 41s |  hbase-hadoop2-compat in the patch 
passed.  |
   | +1 :green_heart: |  unit  |   0m 27s |  hbase-protocol in the patch 
passed.  |
   | +1 :green_heart: |  unit  |   1m 52s |  hbase-client in the patch passed.  
|
   | -1 :x: |  unit  | 163m 15s |  hbase-server in the patch failed.  |
   | +1 :green_heart: |  unit  |   1m  7s |  hbase-hbtop in the patch passed.  |
   | +1 :green_heart: |  asflicense  |   3m 41s |  The patch does not generate 
ASF License warnings.  |
   |  |   | 259m 15s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.5 Server=19.03.5 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/8/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/722 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc prototool |
   | uname | Linux 65e4d40f91f6 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 
11:12:41 UTC 2019 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-722/out/precommit/personality/provided.sh
 |
   | git revision | master / a3efa5911d |
   | Default Java | 1.8.0_181 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/8/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/8/testReport/
 |
   | Max. process+thread count | 4685 (vs. ulimit of 1) |
   | modules | C: hbase-protocol-shaded hbase-hadoop-compat 
hbase-hadoop2-compat hbase-protocol hbase-client hbase-server hbase-hbtop U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/8/console |
   | versions | git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11 |
   | Powered by | Apache Yetus 0.11.1 https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t

[GitHub] [hbase] Apache-HBase commented on issue #722: HBASE-23065 [hbtop] Top-N heavy hitter user and client drill downs

2019-11-17 Thread GitBox
Apache-HBase commented on issue #722: HBASE-23065 [hbtop] Top-N heavy hitter 
user and client drill downs
URL: https://github.com/apache/hbase/pull/722#issuecomment-554814327
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | +0 :ok: |  reexec  |   3m 23s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  1s |  No case conflicting files 
found.  |
   | +0 :ok: |  prototool  |   0m  0s |  prototool was not available.  |
   | +1 :green_heart: |  hbaseanti  |   0m  0s |  Patch does not have any 
anti-patterns.  |
   | +1 :green_heart: |  @author  |   0m  0s |  The patch does not contain any 
@author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  The patch appears to include 
7 new or modified test files.  |
   ||| _ master Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 35s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   5m 40s |  master passed  |
   | +1 :green_heart: |  compile  |   3m 14s |  master passed  |
   | +1 :green_heart: |  checkstyle  |   3m  1s |  master passed  |
   | +1 :green_heart: |  shadedjars  |   5m  0s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   2m  3s |  master passed  |
   | +0 :ok: |  spotbugs  |   4m 28s |  Used deprecated FindBugs config; 
considering switching to SpotBugs.  |
   | +1 :green_heart: |  findbugs  |  10m 57s |  master passed  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 14s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   5m 27s |  the patch passed  |
   | +1 :green_heart: |  compile  |   3m 20s |  the patch passed  |
   | +1 :green_heart: |  cc  |   3m 20s |  the patch passed  |
   | +1 :green_heart: |  javac  |   3m 20s |  the patch passed  |
   | -1 :x: |  checkstyle  |   0m 41s |  hbase-client: The patch generated 2 
new + 0 unchanged - 0 fixed = 2 total (was 0)  |
   | -1 :x: |  checkstyle  |   1m 41s |  hbase-server: The patch generated 4 
new + 254 unchanged - 0 fixed = 258 total (was 254)  |
   | -1 :x: |  checkstyle  |   0m 14s |  hbase-hbtop: The patch generated 5 new 
+ 0 unchanged - 0 fixed = 5 total (was 0)  |
   | +1 :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | +1 :green_heart: |  shadedjars  |   5m 34s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  hadoopcheck  |  17m 51s |  Patch does not cause any 
errors with Hadoop 2.8.5 2.9.2 or 3.1.2.  |
   | +1 :green_heart: |  hbaseprotoc  |   3m 15s |  the patch passed  |
   | +1 :green_heart: |  javadoc  |   2m 15s |  the patch passed  |
   | +1 :green_heart: |  findbugs  |  12m 49s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  |   0m 41s |  hbase-protocol-shaded in the patch 
passed.  |
   | +1 :green_heart: |  unit  |   0m 30s |  hbase-hadoop-compat in the patch 
passed.  |
   | +1 :green_heart: |  unit  |   0m 37s |  hbase-hadoop2-compat in the patch 
passed.  |
   | +1 :green_heart: |  unit  |   0m 26s |  hbase-protocol in the patch 
passed.  |
   | +1 :green_heart: |  unit  |   1m 50s |  hbase-client in the patch passed.  
|
   | -1 :x: |  unit  |  33m 32s |  hbase-server in the patch failed.  |
   | +1 :green_heart: |  unit  |   0m 49s |  hbase-hbtop in the patch passed.  |
   | +1 :green_heart: |  asflicense  |   1m 33s |  The patch does not generate 
ASF License warnings.  |
   |  |   | 136m 27s |   |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.TestClientClusterMetrics |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.5 Server=19.03.5 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/7/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/722 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc prototool |
   | uname | Linux a0d501a746bb 4.15.0-66-generic #75-Ubuntu SMP Tue Oct 1 
05:24:09 UTC 2019 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-722/out/precommit/personality/provided.sh
 |
   | git revision | master / a3efa5911d |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/7/artifact/out/diff-checkstyle-hbase-client.txt
 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/7/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/7/artifact/out/diff-checkstyle-hbase-hbtop.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-72

[GitHub] [hbase] Apache-HBase commented on issue #722: HBASE-23065 [hbtop] Top-N heavy hitter user and client drill downs

2019-10-30 Thread GitBox
Apache-HBase commented on issue #722: HBASE-23065 [hbtop] Top-N heavy hitter 
user and client drill downs
URL: https://github.com/apache/hbase/pull/722#issuecomment-547768520
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | :blue_heart: |  reexec  |   1m  7s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | :green_heart: |  dupname  |   0m  0s |  No case conflicting files found.  |
   | :blue_heart: |  prototool  |   0m  1s |  prototool was not available.  |
   | :green_heart: |  hbaseanti  |   0m  0s |  Patch does not have any 
anti-patterns.  |
   | :green_heart: |  @author  |   0m  0s |  The patch does not contain any 
@author tags.  |
   | :green_heart: |  test4tests  |   0m  0s |  The patch appears to include 7 
new or modified test files.  |
   ||| _ master Compile Tests _ |
   | :blue_heart: |  mvndep  |   0m 33s |  Maven dependency ordering for branch 
 |
   | :green_heart: |  mvninstall  |   5m 47s |  master passed  |
   | :green_heart: |  compile  |   3m 17s |  master passed  |
   | :green_heart: |  checkstyle  |   2m 58s |  master passed  |
   | :green_heart: |  shadedjars  |   4m 58s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | :green_heart: |  javadoc  |   2m  5s |  master passed  |
   | :blue_heart: |  spotbugs  |   4m 25s |  Used deprecated FindBugs config; 
considering switching to SpotBugs.  |
   | :green_heart: |  findbugs  |  10m 57s |  master passed  |
   ||| _ Patch Compile Tests _ |
   | :blue_heart: |  mvndep  |   0m 14s |  Maven dependency ordering for patch  
|
   | :green_heart: |  mvninstall  |   5m 27s |  the patch passed  |
   | :green_heart: |  compile  |   3m 16s |  the patch passed  |
   | :green_heart: |  cc  |   3m 16s |  the patch passed  |
   | :green_heart: |  javac  |   3m 16s |  the patch passed  |
   | :broken_heart: |  checkstyle  |   0m 32s |  hbase-client: The patch 
generated 2 new + 0 unchanged - 0 fixed = 2 total (was 0)  |
   | :broken_heart: |  checkstyle  |   1m 35s |  hbase-server: The patch 
generated 2 new + 273 unchanged - 0 fixed = 275 total (was 273)  |
   | :broken_heart: |  checkstyle  |   0m 12s |  hbase-hbtop: The patch 
generated 5 new + 0 unchanged - 0 fixed = 5 total (was 0)  |
   | :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | :green_heart: |  shadedjars  |   5m  0s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | :green_heart: |  hadoopcheck  |  17m 11s |  Patch does not cause any 
errors with Hadoop 2.8.5 2.9.2 or 3.1.2.  |
   | :green_heart: |  hbaseprotoc  |   3m 13s |  the patch passed  |
   | :green_heart: |  javadoc  |   2m  6s |  the patch passed  |
   | :green_heart: |  findbugs  |  11m 57s |  the patch passed  |
   ||| _ Other Tests _ |
   | :green_heart: |  unit  |   0m 42s |  hbase-protocol-shaded in the patch 
passed.  |
   | :green_heart: |  unit  |   0m 31s |  hbase-hadoop-compat in the patch 
passed.  |
   | :green_heart: |  unit  |   0m 37s |  hbase-hadoop2-compat in the patch 
passed.  |
   | :green_heart: |  unit  |   0m 26s |  hbase-protocol in the patch passed.  |
   | :green_heart: |  unit  |   1m 52s |  hbase-client in the patch passed.  |
   | :broken_heart: |  unit  |  32m  6s |  hbase-server in the patch failed.  |
   | :green_heart: |  unit  |   0m 48s |  hbase-hbtop in the patch passed.  |
   | :green_heart: |  asflicense  |   1m 35s |  The patch does not generate ASF 
License warnings.  |
   |  |   | 130m  9s |   |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.TestClientClusterMetrics |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.4 Server=19.03.4 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/5/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/722 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc prototool |
   | uname | Linux ea948db3f0dd 4.15.0-66-generic #75-Ubuntu SMP Tue Oct 1 
05:24:09 UTC 2019 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-722/out/precommit/personality/provided.sh
 |
   | git revision | master / 2a969e1d8b |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/5/artifact/out/diff-checkstyle-hbase-client.txt
 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/5/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/5/artifact/out/diff-checkstyle-hbase-hbtop.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/5/artifact/out/patch-un

[GitHub] [hbase] Apache-HBase commented on issue #722: HBASE-23065 [hbtop] Top-N heavy hitter user and client drill downs

2019-10-29 Thread GitBox
Apache-HBase commented on issue #722: HBASE-23065 [hbtop] Top-N heavy hitter 
user and client drill downs
URL: https://github.com/apache/hbase/pull/722#issuecomment-547740567
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | :blue_heart: |  reexec  |   1m  9s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | :green_heart: |  dupname  |   0m  0s |  No case conflicting files found.  |
   | :blue_heart: |  prototool  |   0m  0s |  prototool was not available.  |
   | :green_heart: |  hbaseanti  |   0m  0s |  Patch does not have any 
anti-patterns.  |
   | :green_heart: |  @author  |   0m  0s |  The patch does not contain any 
@author tags.  |
   | :green_heart: |  test4tests  |   0m  0s |  The patch appears to include 7 
new or modified test files.  |
   ||| _ master Compile Tests _ |
   | :blue_heart: |  mvndep  |   0m 34s |  Maven dependency ordering for branch 
 |
   | :green_heart: |  mvninstall  |   5m 42s |  master passed  |
   | :green_heart: |  compile  |   3m 18s |  master passed  |
   | :green_heart: |  checkstyle  |   2m 57s |  master passed  |
   | :green_heart: |  shadedjars  |   4m 59s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | :green_heart: |  javadoc  |   2m  5s |  master passed  |
   | :blue_heart: |  spotbugs  |   4m 29s |  Used deprecated FindBugs config; 
considering switching to SpotBugs.  |
   | :green_heart: |  findbugs  |  11m  0s |  master passed  |
   ||| _ Patch Compile Tests _ |
   | :blue_heart: |  mvndep  |   0m 13s |  Maven dependency ordering for patch  
|
   | :green_heart: |  mvninstall  |   5m 24s |  the patch passed  |
   | :green_heart: |  compile  |   3m 14s |  the patch passed  |
   | :green_heart: |  cc  |   3m 14s |  the patch passed  |
   | :green_heart: |  javac  |   3m 14s |  the patch passed  |
   | :broken_heart: |  checkstyle  |   0m 33s |  hbase-client: The patch 
generated 2 new + 0 unchanged - 0 fixed = 2 total (was 0)  |
   | :broken_heart: |  checkstyle  |   1m 32s |  hbase-server: The patch 
generated 2 new + 273 unchanged - 0 fixed = 275 total (was 273)  |
   | :broken_heart: |  checkstyle  |   0m 11s |  hbase-hbtop: The patch 
generated 5 new + 0 unchanged - 0 fixed = 5 total (was 0)  |
   | :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | :green_heart: |  shadedjars  |   5m  2s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | :green_heart: |  hadoopcheck  |  17m 15s |  Patch does not cause any 
errors with Hadoop 2.8.5 2.9.2 or 3.1.2.  |
   | :green_heart: |  hbaseprotoc  |   3m 15s |  the patch passed  |
   | :green_heart: |  javadoc  |   2m  7s |  the patch passed  |
   | :green_heart: |  findbugs  |  11m 57s |  the patch passed  |
   ||| _ Other Tests _ |
   | :green_heart: |  unit  |   0m 41s |  hbase-protocol-shaded in the patch 
passed.  |
   | :green_heart: |  unit  |   0m 31s |  hbase-hadoop-compat in the patch 
passed.  |
   | :green_heart: |  unit  |   0m 38s |  hbase-hadoop2-compat in the patch 
passed.  |
   | :green_heart: |  unit  |   0m 26s |  hbase-protocol in the patch passed.  |
   | :green_heart: |  unit  |   1m 48s |  hbase-client in the patch passed.  |
   | :broken_heart: |  unit  |  32m 48s |  hbase-server in the patch failed.  |
   | :green_heart: |  unit  |   0m 51s |  hbase-hbtop in the patch passed.  |
   | :green_heart: |  asflicense  |   1m 43s |  The patch does not generate ASF 
License warnings.  |
   |  |   | 130m 58s |   |
   
   
   | Reason | Tests |
   |---:|:--|
   | Failed junit tests | hadoop.hbase.TestClientClusterMetrics |
   |   | hadoop.hbase.filter.TestFilterFromRegionSide |
   |   | hadoop.hbase.regionserver.TestKeepDeletes |
   |   | hadoop.hbase.filter.TestInvocationRecordFilter |
   |   | hadoop.hbase.regionserver.TestResettingCounters |
   |   | hadoop.hbase.filter.TestFilter |
   |   | hadoop.hbase.filter.TestDependentColumnFilter |
   |   | hadoop.hbase.regionserver.TestScanner |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.4 Server=19.03.4 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/4/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/722 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc prototool |
   | uname | Linux 329672cece14 4.15.0-66-generic #75-Ubuntu SMP Tue Oct 1 
05:24:09 UTC 2019 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-722/out/precommit/personality/provided.sh
 |
   | git revision | master / 2a969e1d8b |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/4/artifact/out/diff-checkstyle-hbase-client.txt
 |
  

[GitHub] [hbase] Apache-HBase commented on issue #722: HBASE-23065 [hbtop] Top-N heavy hitter user and client drill downs

2019-10-24 Thread GitBox
Apache-HBase commented on issue #722: HBASE-23065 [hbtop] Top-N heavy hitter 
user and client drill downs
URL: https://github.com/apache/hbase/pull/722#issuecomment-545859349
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | :blue_heart: |  reexec  |   0m 38s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | :green_heart: |  dupname  |   0m  1s |  No case conflicting files found.  |
   | :blue_heart: |  prototool  |   0m  0s |  prototool was not available.  |
   | :green_heart: |  hbaseanti  |   0m  0s |  Patch does not have any 
anti-patterns.  |
   | :green_heart: |  @author  |   0m  0s |  The patch does not contain any 
@author tags.  |
   | :green_heart: |  test4tests  |   0m  0s |  The patch appears to include 6 
new or modified test files.  |
   ||| _ master Compile Tests _ |
   | :blue_heart: |  mvndep  |   0m 38s |  Maven dependency ordering for branch 
 |
   | :green_heart: |  mvninstall  |   5m 47s |  master passed  |
   | :green_heart: |  compile  |   3m 23s |  master passed  |
   | :green_heart: |  checkstyle  |   2m 54s |  master passed  |
   | :green_heart: |  shadedjars  |   4m 36s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | :green_heart: |  javadoc  |   2m 18s |  master passed  |
   | :blue_heart: |  spotbugs  |   4m  9s |  Used deprecated FindBugs config; 
considering switching to SpotBugs.  |
   | :green_heart: |  findbugs  |  10m 27s |  master passed  |
   ||| _ Patch Compile Tests _ |
   | :blue_heart: |  mvndep  |   0m 15s |  Maven dependency ordering for patch  
|
   | :green_heart: |  mvninstall  |   4m 55s |  the patch passed  |
   | :green_heart: |  compile  |   3m 23s |  the patch passed  |
   | :green_heart: |  cc  |   3m 23s |  the patch passed  |
   | :green_heart: |  javac  |   3m 23s |  the patch passed  |
   | :broken_heart: |  checkstyle  |   0m 33s |  hbase-client: The patch 
generated 4 new + 0 unchanged - 0 fixed = 4 total (was 0)  |
   | :broken_heart: |  checkstyle  |   1m 19s |  hbase-server: The patch 
generated 4 new + 63 unchanged - 0 fixed = 67 total (was 63)  |
   | :broken_heart: |  checkstyle  |   0m 15s |  hbase-hbtop: The patch 
generated 29 new + 0 unchanged - 0 fixed = 29 total (was 0)  |
   | :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | :green_heart: |  shadedjars  |   4m 39s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | :green_heart: |  hadoopcheck  |  15m 38s |  Patch does not cause any 
errors with Hadoop 2.8.5 2.9.2 or 3.1.2.  |
   | :green_heart: |  hbaseprotoc  |   3m 22s |  the patch passed  |
   | :green_heart: |  javadoc  |   2m 16s |  the patch passed  |
   | :green_heart: |  findbugs  |  11m 23s |  the patch passed  |
   ||| _ Other Tests _ |
   | :green_heart: |  unit  |   0m 40s |  hbase-protocol-shaded in the patch 
passed.  |
   | :green_heart: |  unit  |   0m 34s |  hbase-hadoop-compat in the patch 
passed.  |
   | :green_heart: |  unit  |   0m 40s |  hbase-hadoop2-compat in the patch 
passed.  |
   | :green_heart: |  unit  |   0m 28s |  hbase-protocol in the patch passed.  |
   | :green_heart: |  unit  |   1m 49s |  hbase-client in the patch passed.  |
   | :broken_heart: |  unit  | 158m 43s |  hbase-server in the patch failed.  |
   | :green_heart: |  unit  |   1m  1s |  hbase-hbtop in the patch passed.  |
   | :green_heart: |  asflicense  |   3m 20s |  The patch does not generate ASF 
License warnings.  |
   |  |   | 255m 50s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.4 Server=19.03.4 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/3/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/722 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc prototool |
   | uname | Linux 56e7a232a6d1 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 
11:12:41 UTC 2019 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-722/out/precommit/personality/provided.sh
 |
   | git revision | master / b14ba58c5a |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/3/artifact/out/diff-checkstyle-hbase-client.txt
 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/3/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/3/artifact/out/diff-checkstyle-hbase-hbtop.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/3/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/3/testR

[GitHub] [hbase] Apache-HBase commented on issue #722: HBASE-23065 [hbtop] Top-N heavy hitter user and client drill downs

2019-10-15 Thread GitBox
Apache-HBase commented on issue #722: HBASE-23065 [hbtop] Top-N heavy hitter 
user and client drill downs
URL: https://github.com/apache/hbase/pull/722#issuecomment-542144819
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | :blue_heart: |  reexec  |   1m 13s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | :green_heart: |  dupname  |   0m  0s |  No case conflicting files found.  |
   | :blue_heart: |  prototool  |   0m  0s |  prototool was not available.  |
   | :green_heart: |  hbaseanti  |   0m  0s |  Patch does not have any 
anti-patterns.  |
   | :green_heart: |  @author  |   0m  0s |  The patch does not contain any 
@author tags.  |
   | :green_heart: |  test4tests  |   0m  0s |  The patch appears to include 8 
new or modified test files.  |
   ||| _ master Compile Tests _ |
   | :blue_heart: |  mvndep  |   0m 40s |  Maven dependency ordering for branch 
 |
   | :green_heart: |  mvninstall  |   6m 10s |  master passed  |
   | :green_heart: |  compile  |   3m 19s |  master passed  |
   | :green_heart: |  checkstyle  |   2m 55s |  master passed  |
   | :green_heart: |  shadedjars  |   5m 37s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | :green_heart: |  javadoc  |   2m 26s |  master passed  |
   | :blue_heart: |  spotbugs  |   4m 46s |  Used deprecated FindBugs config; 
considering switching to SpotBugs.  |
   | :green_heart: |  findbugs  |  12m  0s |  master passed  |
   ||| _ Patch Compile Tests _ |
   | :blue_heart: |  mvndep  |   0m 14s |  Maven dependency ordering for patch  
|
   | :green_heart: |  mvninstall  |   5m 28s |  the patch passed  |
   | :green_heart: |  compile  |   3m 15s |  the patch passed  |
   | :green_heart: |  cc  |   3m 15s |  the patch passed  |
   | :green_heart: |  javac  |   3m 15s |  the patch passed  |
   | :broken_heart: |  checkstyle  |   0m 34s |  hbase-client: The patch 
generated 4 new + 0 unchanged - 0 fixed = 4 total (was 0)  |
   | :broken_heart: |  checkstyle  |   1m 27s |  hbase-server: The patch 
generated 2 new + 62 unchanged - 0 fixed = 64 total (was 62)  |
   | :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | :green_heart: |  shadedjars  |   5m  0s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | :green_heart: |  hadoopcheck  |  17m 13s |  Patch does not cause any 
errors with Hadoop 2.8.5 2.9.2 or 3.1.2.  |
   | :green_heart: |  hbaseprotoc  |   3m 16s |  the patch passed  |
   | :green_heart: |  javadoc  |   2m  5s |  the patch passed  |
   | :green_heart: |  findbugs  |  12m  3s |  the patch passed  |
   ||| _ Other Tests _ |
   | :green_heart: |  unit  |   0m 42s |  hbase-protocol-shaded in the patch 
passed.  |
   | :green_heart: |  unit  |   0m 30s |  hbase-hadoop-compat in the patch 
passed.  |
   | :green_heart: |  unit  |   0m 37s |  hbase-hadoop2-compat in the patch 
passed.  |
   | :green_heart: |  unit  |   0m 26s |  hbase-protocol in the patch passed.  |
   | :green_heart: |  unit  |   1m 45s |  hbase-client in the patch passed.  |
   | :broken_heart: |  unit  | 255m  8s |  hbase-server in the patch failed.  |
   | :green_heart: |  unit  |   1m  2s |  hbase-hbtop in the patch passed.  |
   | :green_heart: |  asflicense  |   2m 51s |  The patch does not generate ASF 
License warnings.  |
   |  |   | 357m 14s |   |
   
   
   | Subsystem | Report/Notes |
   |--:|:-|
   | Docker | Client=19.03.3 Server=19.03.3 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/2/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/722 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc prototool |
   | uname | Linux 2c2dc8cad140 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 
16:55:30 UTC 2019 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-722/out/precommit/personality/provided.sh
 |
   | git revision | master / 7924ba39e7 |
   | Default Java | 1.8.0_181 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/2/artifact/out/diff-checkstyle-hbase-client.txt
 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/2/artifact/out/diff-checkstyle-hbase-server.txt
 |
   | unit | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/2/artifact/out/patch-unit-hbase-server.txt
 |
   |  Test Results | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-722/2/testReport/
 |
   | Max. process+thread count | 4777 (vs. ulimit of 1) |
   | modules | C: hbase-protocol-shaded hbase-hadoop-compat 
hbase-hadoop2-compat hbase-protocol hbase-client hbase-server hbase-hbtop U: . |
   | Console output | 
https://builds.apache.org/job/HBas

[GitHub] [hbase] Apache-HBase commented on issue #722: HBASE-23065 [hbtop] Top-N heavy hitter user and client drill downs

2019-10-14 Thread GitBox
Apache-HBase commented on issue #722: HBASE-23065 [hbtop] Top-N heavy hitter 
user and client drill downs
URL: https://github.com/apache/hbase/pull/722#issuecomment-542000506
 
 
   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |::|--:|:|:|
   | :blue_heart: |  reexec  |   2m 22s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | :green_heart: |  dupname  |   0m  0s |  No case conflicting files found.  |
   | :blue_heart: |  prototool  |   0m  0s |  prototool was not available.  |
   | :green_heart: |  hbaseanti  |   0m  0s |  Patch does not have any 
anti-patterns.  |
   | :green_heart: |  @author  |   0m  0s |  The patch does not contain any 
@author tags.  |
   | :green_heart: |  test4tests  |   0m  0s |  The patch appears to include 8 
new or modified test files.  |
   ||| _ master Compile Tests _ |
   | :blue_heart: |  mvndep  |   0m 45s |  Maven dependency ordering for branch 
 |
   | :green_heart: |  mvninstall  |   7m 31s |  master passed  |
   | :green_heart: |  compile  |   3m 39s |  master passed  |
   | :green_heart: |  checkstyle  |   3m 15s |  master passed  |
   | :green_heart: |  shadedjars  |   5m 43s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | :green_heart: |  javadoc  |   2m 41s |  master passed  |
   | :blue_heart: |  spotbugs  |   5m  9s |  Used deprecated FindBugs config; 
considering switching to SpotBugs.  |
   | :green_heart: |  findbugs  |  12m 10s |  master passed  |
   ||| _ Patch Compile Tests _ |
   | :blue_heart: |  mvndep  |   0m 21s |  Maven dependency ordering for patch  
|
   | :green_heart: |  mvninstall  |   6m 39s |  the patch passed  |
   | :green_heart: |  compile  |   3m 33s |  the patch passed  |
   | :green_heart: |  cc  |   3m 33s |  the patch passed  |
   | :green_heart: |  javac  |   3m 33s |  the patch passed  |
   | :broken_heart: |  checkstyle  |   0m 14s |  hbase-hadoop-compat: The patch 
generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)  |
   | :broken_heart: |  checkstyle  |   0m 13s |  hbase-hadoop2-compat: The 
patch generated 3 new + 0 unchanged - 0 fixed = 3 total (was 0)  |
   | :broken_heart: |  checkstyle  |   0m 34s |  hbase-client: The patch 
generated 12 new + 0 unchanged - 0 fixed = 12 total (was 0)  |
   | :broken_heart: |  checkstyle  |   1m 32s |  hbase-server: The patch 
generated 16 new + 62 unchanged - 0 fixed = 78 total (was 62)  |
   | :broken_heart: |  checkstyle  |   0m 13s |  hbase-hbtop: The patch 
generated 38 new + 0 unchanged - 0 fixed = 38 total (was 0)  |
   | :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | :green_heart: |  shadedjars  |   5m  0s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | :green_heart: |  hadoopcheck  |  17m 23s |  Patch does not cause any 
errors with Hadoop 2.8.5 2.9.2 or 3.1.2.  |
   | :green_heart: |  hbaseprotoc  |   3m 15s |  the patch passed  |
   | :broken_heart: |  javadoc  |   0m 15s |  hbase-hadoop-compat generated 2 
new + 0 unchanged - 0 fixed = 2 total (was 0)  |
   | :broken_heart: |  javadoc  |   0m 36s |  hbase-server generated 1 new + 0 
unchanged - 0 fixed = 1 total (was 0)  |
   | :broken_heart: |  javadoc  |   0m 15s |  hbase-hbtop generated 8 new + 0 
unchanged - 0 fixed = 8 total (was 0)  |
   | :broken_heart: |  findbugs  |   0m 39s |  hbase-hadoop2-compat generated 1 
new + 0 unchanged - 0 fixed = 1 total (was 0)  |
   | :broken_heart: |  findbugs  |   1m 17s |  hbase-client generated 2 new + 0 
unchanged - 0 fixed = 2 total (was 0)  |
   | :broken_heart: |  findbugs  |   4m 44s |  hbase-server generated 1 new + 0 
unchanged - 0 fixed = 1 total (was 0)  |
   ||| _ Other Tests _ |
   | :green_heart: |  unit  |   0m 42s |  hbase-protocol-shaded in the patch 
passed.  |
   | :green_heart: |  unit  |   0m 31s |  hbase-hadoop-compat in the patch 
passed.  |
   | :green_heart: |  unit  |   0m 37s |  hbase-hadoop2-compat in the patch 
passed.  |
   | :green_heart: |  unit  |   0m 25s |  hbase-protocol in the patch passed.  |
   | :green_heart: |  unit  |   1m 50s |  hbase-client in the patch passed.  |
   | :broken_heart: |  unit  |  30m 50s |  hbase-server in the patch failed.  |
   | :green_heart: |  unit  |   0m 47s |  hbase-hbtop in the patch passed.  |
   | :green_heart: |  asflicense  |   1m 37s |  The patch does not generate ASF 
License warnings.  |
   |  |   | 137m 31s |   |
   
   
   | Reason | Tests |
   |---:|:--|
   | FindBugs | module:hbase-hadoop2-compat |
   |  |  Should 
org.apache.hadoop.hbase.regionserver.MetricsUserSourceImpl$ClientMetricsImpl be 
a _static_ inner class?  At MetricsUserSourceImpl.java:inner class?  At 
MetricsUserSourceImpl.java:[lines 72-96] |
   | FindBugs | module:hbase-client |
   |  |  Unused field:UserMetricsBuilder.java |
   |  |  Unused field:UserMetricsBuilder.java |
   | FindBugs | module:hbase-server |
   |  |  Dead store to metrics