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


   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |   1m 29s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  8s |  No case conflicting files 
found.  |
   | +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.  |
   ||| _ master Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 15s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   4m 33s |  master passed  |
   | +1 :green_heart: |  compile  |  15m  5s |  master passed  |
   | +1 :green_heart: |  checkstyle  |   8m 48s |  master passed  |
   | +1 :green_heart: |  spotbugs  |  16m  3s |  master passed  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 13s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   4m 28s |  the patch passed  |
   | +1 :green_heart: |  compile  |  15m 43s |  the patch passed  |
   | -0 :warning: |  javac  |   0m 49s |  hbase-common generated 5 new + 155 
unchanged - 4 fixed = 160 total (was 159)  |
   | -0 :warning: |  javac  |   0m 28s |  hbase-zookeeper generated 1 new + 96 
unchanged - 1 fixed = 97 total (was 97)  |
   | -0 :warning: |  javac  |   3m 34s |  hbase-server generated 97 new + 96 
unchanged - 97 fixed = 193 total (was 193)  |
   | -0 :warning: |  javac  |   0m 29s |  hbase-testing-util generated 100 new 
+ 1 unchanged - 0 fixed = 101 total (was 1)  |
   | -0 :warning: |  checkstyle  |   1m 37s |  hbase-server: The patch 
generated 175 new + 1315 unchanged - 173 fixed = 1490 total (was 1488)  |
   | -0 :warning: |  checkstyle  |   0m 18s |  hbase-testing-util: The patch 
generated 174 new + 0 unchanged - 0 fixed = 174 total (was 0)  |
   | -0 :warning: |  whitespace  |   0m  0s |  The patch has 2 line(s) that end 
in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer 
https://git-scm.com/docs/git-apply  |
   | +1 :green_heart: |  xml  |   0m  2s |  The patch has no ill-formed XML 
file.  |
   | +1 :green_heart: |  hadoopcheck  |  20m 46s |  Patch does not cause any 
errors with Hadoop 3.1.2 3.2.1 3.3.0.  |
   | -1 :x: |  spotbugs  |   1m  2s |  hbase-testing-util generated 4 new + 0 
unchanged - 0 fixed = 4 total (was 0)  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  asflicense  |   4m 20s |  The patch does not generate 
ASF License warnings.  |
   |  |   | 138m 33s |   |
   
   
   | Reason | Tests |
   |-------:|:------|
   | FindBugs | module:hbase-testing-util |
   |  |  Inconsistent synchronization of 
org.apache.hadoop.hbase.HBaseTestingUtility.hbaseAdmin; locked 42% of time  
Unsynchronized access at HBaseTestingUtility.java:42% of time  Unsynchronized 
access at HBaseTestingUtility.java:[line 3045] |
   |  |  
org.apache.hadoop.hbase.HBaseTestingUtility.explainTableAvailability(TableName) 
concatenates strings using + in a loop  At HBaseTestingUtility.java:in a loop  
At HBaseTestingUtility.java:[line 3225] |
   |  |  Write to static field 
org.apache.hadoop.hbase.HBaseTestingUtility.FS_URI from instance method 
org.apache.hadoop.hbase.HBaseTestingUtility.setFileSystemURI(String)  At 
HBaseTestingUtility.java:from instance method 
org.apache.hadoop.hbase.HBaseTestingUtility.setFileSystemURI(String)  At 
HBaseTestingUtility.java:[line 4044] |
   |  |  Write to static field org.apache.hadoop.hbase.MiniHBaseCluster.index 
from instance method org.apache.hadoop.hbase.MiniHBaseCluster.startMaster()  At 
MiniHBaseCluster.java:from instance method 
org.apache.hadoop.hbase.MiniHBaseCluster.startMaster()  At 
MiniHBaseCluster.java:[line 551] |
   
   
   | 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-3478/5/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/3478 |
   | Optional Tests | dupname asflicense javac spotbugs hadoopcheck hbaseanti 
checkstyle compile xml |
   | uname | Linux 475fdecf635a 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 
06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 83d1bf1667 |
   | Default Java | AdoptOpenJDK-1.8.0_282-b08 |
   | javac | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3478/5/artifact/yetus-general-check/output/diff-compile-javac-hbase-common.txt
 |
   | javac | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3478/5/artifact/yetus-general-check/output/diff-compile-javac-hbase-zookeeper.txt
 |
   | javac | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3478/5/artifact/yetus-general-check/output/diff-compile-javac-hbase-server.txt
 |
   | javac | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3478/5/artifact/yetus-general-check/output/diff-compile-javac-hbase-testing-util.txt
 |
   | checkstyle | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3478/5/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt
 |
   | checkstyle | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3478/5/artifact/yetus-general-check/output/diff-checkstyle-hbase-testing-util.txt
 |
   | whitespace | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3478/5/artifact/yetus-general-check/output/whitespace-eol.txt
 |
   | spotbugs | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3478/5/artifact/yetus-general-check/output/new-spotbugs-hbase-testing-util.html
 |
   | Max. process+thread count | 86 (vs. ulimit of 30000) |
   | modules | C: hbase-common hbase-client hbase-zookeeper hbase-replication 
hbase-balancer hbase-http hbase-asyncfs hbase-procedure hbase-server 
hbase-mapreduce hbase-testing-util hbase-thrift hbase-shell hbase-endpoint 
hbase-backup hbase-it hbase-rest hbase-examples 
hbase-shaded/hbase-shaded-testing-util-tester 
hbase-archetypes/hbase-client-project 
hbase-archetypes/hbase-shaded-client-project U: . |
   | Console output | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3478/5/console
 |
   | versions | git=2.17.1 maven=3.6.3 spotbugs=4.2.2 |
   | 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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to