Apache-HBase commented on pull request #2940:
URL: https://github.com/apache/hbase/pull/2940#issuecomment-775131624


   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |  12m 51s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  1s |  No case conflicting files 
found.  |
   | +0 :ok: |  jshint  |   0m  1s |  jshint 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 
9 new or modified test files.  |
   ||| _ branch-1 Compile Tests _ |
   | +0 :ok: |  mvndep  |   2m 50s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  14m 37s |  branch-1 passed  |
   | +1 :green_heart: |  compile  |   0m 43s |  branch-1 passed with JDK Azul 
Systems, Inc.-1.8.0_262-b19  |
   | +1 :green_heart: |  compile  |   0m 59s |  branch-1 passed with JDK Azul 
Systems, Inc.-1.7.0_272-b10  |
   | +1 :green_heart: |  checkstyle  |   1m 10s |  branch-1 passed  |
   | +1 :green_heart: |  shadedjars  |   3m 21s |  branch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  javadoc  |   0m 51s |  branch-1 passed with JDK Azul 
Systems, Inc.-1.8.0_262-b19  |
   | +1 :green_heart: |  javadoc  |   2m  7s |  branch-1 passed with JDK Azul 
Systems, Inc.-1.7.0_272-b10  |
   | +0 :ok: |  spotbugs  |   1m 49s |  Used deprecated FindBugs config; 
considering switching to SpotBugs.  |
   | +1 :green_heart: |  findbugs  |   3m 11s |  branch-1 passed  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 17s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   2m  8s |  the patch passed  |
   | +1 :green_heart: |  compile  |   0m 44s |  the patch passed with JDK Azul 
Systems, Inc.-1.8.0_262-b19  |
   | +1 :green_heart: |  javac  |   0m 44s |  the patch passed  |
   | +1 :green_heart: |  compile  |   0m 57s |  the patch passed with JDK Azul 
Systems, Inc.-1.7.0_272-b10  |
   | -1 :x: |  javac  |   0m 34s |  
hbase-thrift-jdkAzulSystems,Inc.-1.7.0_272-b10 with JDK Azul Systems, 
Inc.-1.7.0_272-b10 generated 34 new + 71 unchanged - 31 fixed = 105 total (was 
102)  |
   | -1 :x: |  checkstyle  |   0m 32s |  hbase-common: The patch generated 1 
new + 0 unchanged - 0 fixed = 1 total (was 0)  |
   | -1 :x: |  checkstyle  |   0m 43s |  hbase-thrift: The patch generated 354 
new + 83 unchanged - 76 fixed = 437 total (was 159)  |
   | +1 :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | +1 :green_heart: |  shadedjars  |   3m  7s |  patch has no errors when 
building our shaded downstream artifacts.  |
   | +1 :green_heart: |  hadoopcheck  |   5m  4s |  Patch does not cause any 
errors with Hadoop 2.8.5 2.9.2.  |
   | -1 :x: |  javadoc  |   0m 29s |  
hbase-thrift-jdkAzulSystems,Inc.-1.8.0_262-b19 with JDK Azul Systems, 
Inc.-1.8.0_262-b19 generated 13 new + 0 unchanged - 0 fixed = 13 total (was 0)  
|
   | -1 :x: |  javadoc  |   3m 32s |  
hbase-thrift-jdkAzulSystems,Inc.-1.7.0_272-b10 with JDK Azul Systems, 
Inc.-1.7.0_272-b10 generated 13 new + 0 unchanged - 0 fixed = 13 total (was 0)  
|
   | -1 :x: |  findbugs  |   2m 31s |  hbase-thrift generated 1 new + 0 
unchanged - 0 fixed = 1 total (was 0)  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  |   3m  2s |  hbase-common in the patch passed.  
|
   | -1 :x: |  unit  |   0m 38s |  hbase-thrift in the patch failed.  |
   | +1 :green_heart: |  asflicense  |   0m 27s |  The patch does not generate 
ASF License warnings.  |
   |  |   |  74m 30s |   |
   
   
   | Reason | Tests |
   |-------:|:------|
   | FindBugs | module:hbase-thrift |
   |  |  The class name org.apache.hadoop.hbase.thrift2.ThriftServer shadows 
the simple name of the superclass org.apache.hadoop.hbase.thrift.ThriftServer  
At ThriftServer.java:the simple name of the superclass 
org.apache.hadoop.hbase.thrift.ThriftServer  At ThriftServer.java:[lines 
52-113] |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2940/1/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/2940 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs 
shadedjars hadoopcheck hbaseanti checkstyle compile jshint |
   | uname | Linux dc8472d3b162 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 
23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | 
/home/jenkins/jenkins-home/workspace/Base-PreCommit-GitHub-PR_PR-2940/out/precommit/personality/provided.sh
 |
   | git revision | branch-1 / 7d80d99 |
   | Default Java | Azul Systems, Inc.-1.7.0_272-b10 |
   | Multi-JDK versions | /usr/lib/jvm/zulu-8-amd64:Azul Systems, 
Inc.-1.8.0_262-b19 /usr/lib/jvm/zulu-7-amd64:Azul Systems, Inc.-1.7.0_272-b10 |
   | javac | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2940/1/artifact/out/diff-compile-javac-hbase-thrift-jdkAzulSystems,Inc.-1.7.0_272-b10.txt
 |
   | checkstyle | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2940/1/artifact/out/diff-checkstyle-hbase-common.txt
 |
   | checkstyle | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2940/1/artifact/out/diff-checkstyle-hbase-thrift.txt
 |
   | javadoc | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2940/1/artifact/out/diff-javadoc-javadoc-hbase-thrift-jdkAzulSystems,Inc.-1.8.0_262-b19.txt
 |
   | javadoc | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2940/1/artifact/out/diff-javadoc-javadoc-hbase-thrift-jdkAzulSystems,Inc.-1.7.0_272-b10.txt
 |
   | findbugs | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2940/1/artifact/out/new-findbugs-hbase-thrift.html
 |
   | unit | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2940/1/artifact/out/patch-unit-hbase-thrift.txt
 |
   |  Test Results | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2940/1/testReport/
 |
   | Max. process+thread count | 171 (vs. ulimit of 10000) |
   | modules | C: hbase-common hbase-thrift U: . |
   | Console output | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2940/1/console
 |
   | versions | git=1.9.1 maven=3.0.5 findbugs=3.0.1 |
   | Powered by | Apache Yetus 0.12.0 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 the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to