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


   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |   1m  5s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  1s |  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.  |
   ||| _ master Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 20s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   4m  4s |  master passed  |
   | +1 :green_heart: |  checkstyle  |   2m 34s |  master passed  |
   | +1 :green_heart: |  spotbugs  |  19m 17s |  master passed  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 13s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   3m 50s |  the patch passed  |
   | -0 :warning: |  checkstyle  |   2m 34s |  root: The patch generated 76 new 
+ 962 unchanged - 5 fixed = 1038 total (was 967)  |
   | -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  |  18m 54s |  Patch does not cause any 
errors with Hadoop 3.1.2 3.2.1 3.3.0.  |
   | +1 :green_heart: |  hbaseprotoc  |   7m 28s |  the patch passed  |
   | -1 :x: |  spotbugs  |   1m 12s |  hbase-client generated 3 new + 0 
unchanged - 0 fixed = 3 total (was 0)  |
   | -1 :x: |  spotbugs  |   2m 18s |  hbase-server generated 1 new + 0 
unchanged - 0 fixed = 1 total (was 0)  |
   | -1 :x: |  spotbugs  |  10m 13s |  root generated 4 new + 0 unchanged - 0 
fixed = 4 total (was 0)  |
   ||| _ Other Tests _ |
   | -1 :x: |  asflicense  |   1m 26s |  The patch generated 2 ASF License 
warnings.  |
   |  |   |  92m 28s |   |
   
   
   | Reason | Tests |
   |-------:|:------|
   | FindBugs | module:hbase-client |
   |  |  Nullcheck of other at line 833 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.isOverlap(RegionInfo)  At 
RegionInfo.java:833 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.isOverlap(RegionInfo)  At 
RegionInfo.java:[line 832] |
   |  |  Nullcheck of rhs at line 113 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.lambda$static$0(RegionInfo, 
RegionInfo)  At RegionInfo.java:113 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.lambda$static$0(RegionInfo, 
RegionInfo)  At RegionInfo.java:[line 112] |
   |  |  Nullcheck of regionA at line 448 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.areAdjacent(RegionInfo, RegionInfo)  
At RegionInfo.java:448 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.areAdjacent(RegionInfo, RegionInfo)  
At RegionInfo.java:[line 447] |
   | 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 833 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.isOverlap(RegionInfo)  At 
RegionInfo.java:833 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.isOverlap(RegionInfo)  At 
RegionInfo.java:[line 832] |
   |  |  Nullcheck of rhs at line 113 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.lambda$static$0(RegionInfo, 
RegionInfo)  At RegionInfo.java:113 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.lambda$static$0(RegionInfo, 
RegionInfo)  At RegionInfo.java:[line 112] |
   |  |  Nullcheck of regionA at line 448 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.areAdjacent(RegionInfo, RegionInfo)  
At RegionInfo.java:448 of value previously dereferenced in 
org.apache.hadoop.hbase.client.RegionInfo.areAdjacent(RegionInfo, RegionInfo)  
At RegionInfo.java:[line 447] |
   |  |  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/5/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 bfcb56fb39ed 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 | dev-support/hbase-personality.sh |
   | git revision | master / e6c78f4037 |
   | checkstyle | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2445/5/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/5/artifact/yetus-general-check/output/whitespace-eol.txt
 |
   | spotbugs | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2445/5/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/5/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/5/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/5/artifact/yetus-general-check/output/patch-asflicense-problems.txt
 |
   | Max. process+thread count | 122 (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/5/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