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


   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |   0m 38s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  2s |  No case conflicting files 
found.  |
   | +0 :ok: |  prototool  |   0m  2s |  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.  |
   ||| _ master Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 21s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   3m 37s |  master passed  |
   | +1 :green_heart: |  checkstyle  |   2m 14s |  master passed  |
   | +1 :green_heart: |  spotbugs  |  17m 16s |  master passed  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 13s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   3m 25s |  the patch passed  |
   | -0 :warning: |  checkstyle  |   2m 19s |  root: The patch generated 68 new 
+ 916 unchanged - 5 fixed = 984 total (was 921)  |
   | -0 :warning: |  whitespace  |   0m  0s |  The patch has 1 line(s) that end 
in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer 
https://git-scm.com/docs/git-apply  |
   | +1 :green_heart: |  hadoopcheck  |  13m 24s |  Patch does not cause any 
errors with Hadoop 3.1.2 3.2.1.  |
   | +1 :green_heart: |  hbaseprotoc  |   7m 24s |  the patch passed  |
   | -1 :x: |  spotbugs  |   1m 14s |  hbase-client generated 3 new + 0 
unchanged - 0 fixed = 3 total (was 0)  |
   | -1 :x: |  spotbugs  |   2m 14s |  hbase-server generated 1 new + 0 
unchanged - 0 fixed = 1 total (was 0)  |
   | -1 :x: |  spotbugs  |   9m 41s |  root generated 4 new + 0 unchanged - 0 
fixed = 4 total (was 0)  |
   ||| _ Other Tests _ |
   | -1 :x: |  asflicense  |   1m 36s |  The patch generated 1 ASF License 
warnings.  |
   |  |   |  83m 27s |   |
   
   
   | Reason | Tests |
   |-------:|:------|
   | FindBugs | module:hbase-client |
   |  |  Nullcheck of other at line 829 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.isOverlap(RegionInfo)  At 
RegionInfo.java:829 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.isOverlap(RegionInfo)  At 
RegionInfo.java:[line 828] |
   |  |  Nullcheck of rhs at line 109 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.lambda$static$0(RegionInfo, 
RegionInfo)  At RegionInfo.java:109 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.lambda$static$0(RegionInfo, 
RegionInfo)  At RegionInfo.java:[line 108] |
   |  |  Nullcheck of regionA at line 444 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.areAdjacent(RegionInfo, RegionInfo)  
At RegionInfo.java:444 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.areAdjacent(RegionInfo, RegionInfo)  
At RegionInfo.java:[line 443] |
   | FindBugs | module:hbase-server |
   |  |  Possible null pointer dereference of region in 
org.apache.hadoop.hbase.regionserver.handler.UnassignRegionHandler.create(HRegionServer,
 String, long, boolean, ServerName)  Dereferenced at 
UnassignRegionHandler.java:region in 
org.apache.hadoop.hbase.regionserver.handler.UnassignRegionHandler.create(HRegionServer,
 String, long, boolean, ServerName)  Dereferenced at 
UnassignRegionHandler.java:[line 149] |
   | FindBugs | module:root |
   |  |  Nullcheck of other at line 829 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.isOverlap(RegionInfo)  At 
RegionInfo.java:829 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.isOverlap(RegionInfo)  At 
RegionInfo.java:[line 828] |
   |  |  Nullcheck of rhs at line 109 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.lambda$static$0(RegionInfo, 
RegionInfo)  At RegionInfo.java:109 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.lambda$static$0(RegionInfo, 
RegionInfo)  At RegionInfo.java:[line 108] |
   |  |  Nullcheck of regionA at line 444 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.areAdjacent(RegionInfo, RegionInfo)  
At RegionInfo.java:444 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.areAdjacent(RegionInfo, RegionInfo)  
At RegionInfo.java:[line 443] |
   |  |  Possible null pointer dereference of region in 
org.apache.hadoop.hbase.regionserver.handler.UnassignRegionHandler.create(HRegionServer,
 String, long, boolean, ServerName)  Dereferenced at 
UnassignRegionHandler.java:region in 
org.apache.hadoop.hbase.regionserver.handler.UnassignRegionHandler.create(HRegionServer,
 String, long, boolean, ServerName)  Dereferenced at 
UnassignRegionHandler.java:[line 149] |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | Client=19.03.13 Server=19.03.13 base: 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2445/1/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/2445 |
   | Optional Tests | dupname asflicense spotbugs hadoopcheck hbaseanti 
checkstyle cc hbaseprotoc prototool |
   | uname | Linux dc508e71d578 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 
16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 687e53b7e4 |
   | checkstyle | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2445/1/artifact/yetus-general-check/output/diff-checkstyle-root.txt
 |
   | whitespace | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2445/1/artifact/yetus-general-check/output/whitespace-eol.txt
 |
   | spotbugs | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2445/1/artifact/yetus-general-check/output/new-spotbugs-hbase-client.html
 |
   | spotbugs | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2445/1/artifact/yetus-general-check/output/new-spotbugs-hbase-server.html
 |
   | spotbugs | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2445/1/artifact/yetus-general-check/output/new-spotbugs-root.html
 |
   | asflicense | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2445/1/artifact/yetus-general-check/output/patch-asflicense-problems.txt
 |
   | Max. process+thread count | 137 (vs. ulimit of 30000) |
   | modules | C: hbase-protocol-shaded hbase-common hbase-client 
hbase-zookeeper hbase-balancer hbase-server hbase-backup . U: . |
   | Console output | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2445/1/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:
us...@infra.apache.org


Reply via email to