Apache-HBase commented on PR #7334:
URL: https://github.com/apache/hbase/pull/7334#issuecomment-3316504471

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |:----:|----------:|--------:|:--------:|:-------:|
   | +0 :ok: |  reexec  |   3m  9s |  |  Docker mode activated.  |
   |||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  0s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  0s |  |  codespell was not available.  |
   | +0 :ok: |  detsecrets  |   0m  0s |  |  detect-secrets was not available.  
|
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | +1 :green_heart: |  hbaseanti  |   0m  0s |  |  Patch does not have any 
anti-patterns.  |
   |||| _ master Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 15s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   3m 18s |  |  master passed  |
   | +1 :green_heart: |  compile  |   8m 36s |  |  master passed  |
   | -0 :warning: |  checkstyle  |   0m 27s | 
[/buildtool-branch-checkstyle-root.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7334/1/artifact/yetus-general-check/output/buildtool-branch-checkstyle-root.txt)
 |  The patch fails to run checkstyle in root  |
   | +1 :green_heart: |  spotbugs  |   9m 13s |  |  master passed  |
   | +1 :green_heart: |  spotless  |   0m 50s |  |  branch has no errors when 
running spotless:check.  |
   |||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 18s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   3m 19s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |   8m 31s |  |  the patch passed  |
   | +1 :green_heart: |  javac  |   8m 31s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   0m 27s | 
[/buildtool-patch-checkstyle-root.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7334/1/artifact/yetus-general-check/output/buildtool-patch-checkstyle-root.txt)
 |  The patch fails to run checkstyle in root  |
   | +1 :green_heart: |  xmllint  |   0m  0s |  |  No new issues.  |
   | -1 :x: |  spotbugs  |   1m 45s | 
[/new-spotbugs-hbase-server.html](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7334/1/artifact/yetus-general-check/output/new-spotbugs-hbase-server.html)
 |  hbase-server generated 28 new + 0 unchanged - 0 fixed = 28 total (was 0)  |
   | -1 :x: |  spotbugs  |   7m 43s | 
[/new-spotbugs-root.html](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7334/1/artifact/yetus-general-check/output/new-spotbugs-root.html)
 |  root generated 28 new + 0 unchanged - 0 fixed = 28 total (was 0)  |
   | +1 :green_heart: |  hadoopcheck  |  13m 11s |  |  Patch does not cause any 
errors with Hadoop 3.3.6 3.4.1.  |
   | +1 :green_heart: |  spotless  |   0m 45s |  |  patch has no errors when 
running spotless:check.  |
   |||| _ Other Tests _ |
   | -1 :x: |  asflicense  |   0m 29s | 
[/results-asflicense.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7334/1/artifact/yetus-general-check/output/results-asflicense.txt)
 |  The patch generated 1 ASF License warnings.  |
   |  |   |  70m 45s |  |  |
   
   
   | Reason | Tests |
   |-------:|:------|
   | SpotBugs | module:hbase-server |
   |  |  org.apache.hadoop.hbase.regionserver.RequestStatsCollector.add(byte[], 
byte[], TableName, Region$Operation, String, long, long, long, long) uses the 
nextDouble method of Random to generate a random integer; using nextInt is more 
efficient  At RequestStatsCollector.java:long, long, long, long) uses the 
nextDouble method of Random to generate a random integer; using nextInt is more 
efficient  At RequestStatsCollector.java:[line 253] |
   |  |  
org.apache.hadoop.hbase.regionserver.RequestStatsCollector.initCalcite() may 
fail to close Connection  At RequestStatsCollector.java:Connection  At 
RequestStatsCollector.java:[line 644] |
   |  |  Unread field:field be static?  At RequestStatsCollector.java:[line 
503] |
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 529] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 527] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 528] 
|
   |  |  Unread field:field be static?  At RequestStatsCollector.java:[line 
604] |
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 628] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 611] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 515] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 516] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 514] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 512] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 518] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 517] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 522] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 521] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 520] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 519] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 511] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 513] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 106] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 104] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 105] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 102] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 101] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 103] 
|
   |  |  HTTP parameter written to Servlet error page in 
org.apache.hadoop.hbase.regionserver.http.RSRequestServlet.doPost(HttpServletRequest,
 HttpServletResponse)  At RSRequestServlet.java:error page in 
org.apache.hadoop.hbase.regionserver.http.RSRequestServlet.doPost(HttpServletRequest,
 HttpServletResponse)  At RSRequestServlet.java:[line 149] |
   | SpotBugs | module:root |
   |  |  org.apache.hadoop.hbase.regionserver.RequestStatsCollector.add(byte[], 
byte[], TableName, Region$Operation, String, long, long, long, long) uses the 
nextDouble method of Random to generate a random integer; using nextInt is more 
efficient  At RequestStatsCollector.java:long, long, long, long) uses the 
nextDouble method of Random to generate a random integer; using nextInt is more 
efficient  At RequestStatsCollector.java:[line 253] |
   |  |  
org.apache.hadoop.hbase.regionserver.RequestStatsCollector.initCalcite() may 
fail to close Connection  At RequestStatsCollector.java:Connection  At 
RequestStatsCollector.java:[line 644] |
   |  |  Unread field:field be static?  At RequestStatsCollector.java:[line 
503] |
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 529] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 527] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 528] 
|
   |  |  Unread field:field be static?  At RequestStatsCollector.java:[line 
604] |
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 628] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 611] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 515] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 516] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 514] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 512] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 518] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 517] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 522] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 521] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 520] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 519] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 511] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 513] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 106] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 104] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 105] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 102] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 101] 
|
   |  |  Unread public/protected field:At RequestStatsCollector.java:[line 103] 
|
   |  |  HTTP parameter written to Servlet error page in 
org.apache.hadoop.hbase.regionserver.http.RSRequestServlet.doPost(HttpServletRequest,
 HttpServletResponse)  At RSRequestServlet.java:error page in 
org.apache.hadoop.hbase.regionserver.http.RSRequestServlet.doPost(HttpServletRequest,
 HttpServletResponse)  At RSRequestServlet.java:[line 149] |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.43 ServerAPI=1.43 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7334/1/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/7334 |
   | Optional Tests | dupname asflicense javac codespell detsecrets xmllint 
spotless hadoopcheck compile spotbugs checkstyle hbaseanti |
   | uname | Linux 91c42802bcec 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 
23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | master / 8ad002d493546babec39768c451a607ed27a64d4 |
   | Default Java | Eclipse Adoptium-17.0.11+9 |
   | Max. process+thread count | 191 (vs. ulimit of 30000) |
   | modules | C: hbase-resource-bundle hbase-server . U: . |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7334/1/console 
|
   | versions | git=2.34.1 maven=3.9.8 spotbugs=4.7.3 xmllint=20913 |
   | Powered by | Apache Yetus 0.15.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