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


   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |   0m 27s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  1s |  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 31s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   3m 52s |  master passed  |
   | +1 :green_heart: |  checkstyle  |   2m 22s |  master passed  |
   | +1 :green_heart: |  spotbugs  |  13m 42s |  master passed  |
   | -0 :warning: |  patch  |   2m 53s |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 12s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   3m 41s |  the patch passed  |
   | -0 :warning: |  checkstyle  |   2m 22s |  root: The patch generated 5 new 
+ 314 unchanged - 31 fixed = 319 total (was 345)  |
   | -0 :warning: |  whitespace  |   0m  0s |  The patch has 9 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  5s |  The patch has no ill-formed XML 
file.  |
   | +1 :green_heart: |  hadoopcheck  |  12m  0s |  Patch does not cause any 
errors with Hadoop 3.1.2 3.2.1.  |
   | -1 :x: |  spotbugs  |   2m 21s |  hbase-server generated 3 new + 0 
unchanged - 0 fixed = 3 total (was 0)  |
   | -1 :x: |  spotbugs  |   9m 44s |  root generated 4 new + 0 unchanged - 0 
fixed = 4 total (was 0)  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  asflicense  |   1m 13s |  The patch does not generate 
ASF License warnings.  |
   |  |   |  65m 22s |   |
   
   
   | Reason | Tests |
   |-------:|:------|
   | FindBugs | module:hbase-server |
   |  |  
org.apache.hadoop.hbase.HBaseTestingUtility.explainTableAvailability(TableName) 
concatenates strings using + in a loop  At HBaseTestingUtility.java:in a loop  
At HBaseTestingUtility.java:[line 3255] |
   |  |  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 4152] |
   |  |  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 546] |
   | FindBugs | module:root |
   |  |  
org.apache.hadoop.hbase.HBaseTestingUtility.explainTableAvailability(TableName) 
concatenates strings using + in a loop  At HBaseTestingUtility.java:in a loop  
At HBaseTestingUtility.java:[line 3255] |
   |  |  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 4152] |
   |  |  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 546] |
   |  |  
org.apache.hadoop.hbase.chaos.factories.MonkeyConstants.MONKEY_CONFIGURATION_KEY_PREFIXES
 is a mutable collection  At MonkeyConstants.java: At 
MonkeyConstants.java:[line 75] |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | Client=19.03.8 Server=19.03.8 base: 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1201/7/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/1201 |
   | Optional Tests | dupname asflicense hadoopcheck xml spotbugs hbaseanti 
checkstyle |
   | uname | Linux a3eca9e0ffd8 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 
08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / e37aafcfc2 |
   | checkstyle | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1201/7/artifact/yetus-general-check/output/diff-checkstyle-root.txt
 |
   | whitespace | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1201/7/artifact/yetus-general-check/output/whitespace-eol.txt
 |
   | spotbugs | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1201/7/artifact/yetus-general-check/output/new-spotbugs-hbase-server.html
 |
   | spotbugs | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1201/7/artifact/yetus-general-check/output/new-spotbugs-root.html
 |
   | Max. process+thread count | 123 (vs. ulimit of 12500) |
   | modules | C: hbase-common hbase-hadoop-compat hbase-zookeeper hbase-http 
hbase-server hbase-it . U: . |
   | Console output | 
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1201/7/console |
   | versions | git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) 
spotbugs=3.1.12 |
   | 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 the
URL above to go to the specific comment.

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


Reply via email to