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


   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |   1m 21s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  1s |  No case conflicting files 
found.  |
   | +0 :ok: |  prototool  |   0m  0s |  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.  |
   ||| _ HBASE-26067 Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 15s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   4m 41s |  HBASE-26067 passed  |
   | +1 :green_heart: |  compile  |   7m 14s |  HBASE-26067 passed  |
   | +1 :green_heart: |  checkstyle  |   2m 21s |  HBASE-26067 passed  |
   | +1 :green_heart: |  spotbugs  |   9m 26s |  HBASE-26067 passed  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 12s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   4m 26s |  the patch passed  |
   | +1 :green_heart: |  compile  |   7m 46s |  the patch passed  |
   | +1 :green_heart: |  cc  |   7m 46s |  the patch passed  |
   | -0 :warning: |  javac  |   0m 32s |  hbase-hadoop-compat generated 6 new + 
97 unchanged - 6 fixed = 103 total (was 103)  |
   | -0 :warning: |  javac  |   1m  4s |  hbase-client generated 1 new + 121 
unchanged - 1 fixed = 122 total (was 122)  |
   | -0 :warning: |  checkstyle  |   0m 16s |  hbase-hadoop-compat: The patch 
generated 14 new + 0 unchanged - 0 fixed = 14 total (was 0)  |
   | -0 :warning: |  checkstyle  |   0m 35s |  hbase-client: The patch 
generated 5 new + 181 unchanged - 0 fixed = 186 total (was 181)  |
   | -0 :warning: |  checkstyle  |   1m 19s |  hbase-server: The patch 
generated 27 new + 162 unchanged - 0 fixed = 189 total (was 162)  |
   | -0 :warning: |  checkstyle  |   0m 17s |  hbase-rest: The patch generated 
3 new + 3 unchanged - 0 fixed = 6 total (was 3)  |
   | +1 :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace 
issues.  |
   | +1 :green_heart: |  hadoopcheck  |  21m 11s |  Patch does not cause any 
errors with Hadoop 3.1.2 3.2.1 3.3.0.  |
   | +1 :green_heart: |  hbaseprotoc  |   2m 48s |  the patch passed  |
   | -1 :x: |  spotbugs  |   2m 28s |  hbase-server generated 2 new + 0 
unchanged - 0 fixed = 2 total (was 0)  |
   | -1 :x: |  spotbugs  |   0m 54s |  hbase-rest generated 1 new + 0 unchanged 
- 0 fixed = 1 total (was 0)  |
   ||| _ Other Tests _ |
   | -1 :x: |  asflicense  |   0m 53s |  The patch generated 1 ASF License 
warnings.  |
   |  |   |  84m 52s |   |
   
   
   | Reason | Tests |
   |-------:|:------|
   | FindBugs | module:hbase-server |
   |  |  Dead store to fsStoreFiles in 
org.apache.hadoop.hbase.regionserver.FileBasedStoreFileCleaner.chore()  At 
FileBasedStoreFileCleaner.java:org.apache.hadoop.hbase.regionserver.FileBasedStoreFileCleaner.chore()
  At FileBasedStoreFileCleaner.java:[line 94] |
   |  |  Random object created and used only once in 
org.apache.hadoop.hbase.regionserver.HRegionServer.initializeThreads()  At 
HRegionServer.java:only once in 
org.apache.hadoop.hbase.regionserver.HRegionServer.initializeThreads()  At 
HRegionServer.java:[line 1990] |
   | FindBugs | module:hbase-rest |
   |  |  Class 
org.apache.hadoop.hbase.rest.model.FileBasedStoreFileCleanerStatusModel defines 
non-transient non-serializable instance field fileBasedFileStoreCleanerStatus  
In FileBasedStoreFileCleanerStatusModel.java:instance field 
fileBasedFileStoreCleanerStatus  In FileBasedStoreFileCleanerStatusModel.java |
   
   
   | 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-3751/1/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/3751 |
   | JIRA Issue | HBASE-26271 |
   | Optional Tests | dupname asflicense javac spotbugs hadoopcheck hbaseanti 
checkstyle compile cc hbaseprotoc prototool |
   | uname | Linux 31ea4f640230 4.15.0-147-generic #151-Ubuntu SMP Fri Jun 18 
19:21:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | HBASE-26067 / 9c9789bbd2 |
   | Default Java | AdoptOpenJDK-1.8.0_282-b08 |
   | javac | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3751/1/artifact/yetus-general-check/output/diff-compile-javac-hbase-hadoop-compat.txt
 |
   | javac | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3751/1/artifact/yetus-general-check/output/diff-compile-javac-hbase-client.txt
 |
   | checkstyle | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3751/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-hadoop-compat.txt
 |
   | checkstyle | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3751/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-client.txt
 |
   | checkstyle | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3751/1/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-3751/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-rest.txt
 |
   | spotbugs | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3751/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-3751/1/artifact/yetus-general-check/output/new-spotbugs-hbase-rest.html
 |
   | asflicense | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3751/1/artifact/yetus-general-check/output/patch-asflicense-problems.txt
 |
   | Max. process+thread count | 87 (vs. ulimit of 30000) |
   | modules | C: hbase-protocol-shaded hbase-hadoop-compat hbase-client 
hbase-server hbase-rest U: . |
   | Console output | 
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3751/1/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: issues-unsubscr...@hbase.apache.org

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


Reply via email to