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

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |:----:|----------:|--------:|:--------:|:-------:|
   | +0 :ok: |  reexec  |   0m 13s |  |  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.  
|
   | +0 :ok: |  buf  |   0m  0s |  |  buf was not available.  |
   | +0 :ok: |  buf  |   0m  0s |  |  buf 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.  |
   |||| _ HBASE-29368-key-management-feature Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 19s |  |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   8m  6s |  |  
HBASE-29368-key-management-feature passed  |
   | +1 :green_heart: |  compile  |  11m  3s |  |  
HBASE-29368-key-management-feature passed  |
   | +1 :green_heart: |  checkstyle  |   3m 58s |  |  
HBASE-29368-key-management-feature passed  |
   | +1 :green_heart: |  spotbugs  |  13m 38s |  |  
HBASE-29368-key-management-feature passed  |
   | +1 :green_heart: |  spotless  |   1m 41s |  |  branch has no errors when 
running spotless:check.  |
   |||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 20s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   7m 37s |  |  the patch passed  |
   | -1 :x: |  compile  |   1m 11s | 
[/patch-compile-hbase-common.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7460/6/artifact/yetus-general-check/output/patch-compile-hbase-common.txt)
 |  hbase-common in the patch failed.  |
   | -1 :x: |  compile  |   1m 37s | 
[/patch-compile-hbase-server.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7460/6/artifact/yetus-general-check/output/patch-compile-hbase-server.txt)
 |  hbase-server in the patch failed.  |
   | -1 :x: |  cc  |   1m 11s | 
[/patch-compile-hbase-common.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7460/6/artifact/yetus-general-check/output/patch-compile-hbase-common.txt)
 |  hbase-common in the patch failed.  |
   | -1 :x: |  cc  |   1m 37s | 
[/patch-compile-hbase-server.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7460/6/artifact/yetus-general-check/output/patch-compile-hbase-server.txt)
 |  hbase-server in the patch failed.  |
   | -0 :warning: |  javac  |   1m 11s | 
[/patch-compile-hbase-common.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7460/6/artifact/yetus-general-check/output/patch-compile-hbase-common.txt)
 |  hbase-common in the patch failed.  |
   | -0 :warning: |  javac  |   1m 37s | 
[/patch-compile-hbase-server.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7460/6/artifact/yetus-general-check/output/patch-compile-hbase-server.txt)
 |  hbase-server in the patch failed.  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   1m 12s | 
[/results-checkstyle-hbase-server.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7460/6/artifact/yetus-general-check/output/results-checkstyle-hbase-server.txt)
 |  hbase-server: The patch generated 12 new + 9 unchanged - 1 fixed = 21 total 
(was 10)  |
   | -0 :warning: |  rubocop  |   0m 21s | 
[/results-rubocop.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7460/6/artifact/yetus-general-check/output/results-rubocop.txt)
 |  The patch generated 15 new + 44 unchanged - 1 fixed = 59 total (was 45)  |
   | -1 :x: |  spotbugs  |   0m 59s | 
[/new-spotbugs-hbase-common.html](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7460/6/artifact/yetus-general-check/output/new-spotbugs-hbase-common.html)
 |  hbase-common generated 2 new + 0 unchanged - 0 fixed = 2 total (was 0)  |
   | -1 :x: |  spotbugs  |   3m  4s | 
[/new-spotbugs-hbase-server.html](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7460/6/artifact/yetus-general-check/output/new-spotbugs-hbase-server.html)
 |  hbase-server generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)  |
   | +1 :green_heart: |  hadoopcheck  |  12m 18s |  |  Patch does not cause any 
errors with Hadoop 3.3.6 3.4.1.  |
   | +1 :green_heart: |  hbaseprotoc  |   2m 32s |  |  the patch passed  |
   | +1 :green_heart: |  spotless  |   0m 48s |  |  patch has no errors when 
running spotless:check.  |
   |||| _ Other Tests _ |
   | +1 :green_heart: |  asflicense  |   0m 50s |  |  The patch does not 
generate ASF License warnings.  |
   |  |   |  92m 56s |  |  |
   
   
   | Reason | Tests |
   |-------:|:------|
   | SpotBugs | module:hbase-common |
   |  |  Nullcheck of keyMetadata at line 116 of value previously dereferenced 
in new org.apache.hadoop.hbase.io.crypto.ManagedKeyData(byte[], String, Key, 
ManagedKeyState, String, long)  At ManagedKeyData.java:116 of value previously 
dereferenced in new org.apache.hadoop.hbase.io.crypto.ManagedKeyData(byte[], 
String, Key, ManagedKeyState, String, long)  At ManagedKeyData.java:[line 107] |
   |  |  Uninitialized read of keyMetadataHash in new 
org.apache.hadoop.hbase.io.crypto.ManagedKeyData(byte[], String, Key, 
ManagedKeyState, String, long)  At ManagedKeyData.java:new 
org.apache.hadoop.hbase.io.crypto.ManagedKeyData(byte[], String, Key, 
ManagedKeyState, String, long)  At ManagedKeyData.java:[line 114] |
   | SpotBugs | module:hbase-server |
   |  |  Dead store to markerKey in 
org.apache.hadoop.hbase.keymeta.KeymetaAdminImpl.disableKeyManagement(byte[], 
String)  At 
KeymetaAdminImpl.java:org.apache.hadoop.hbase.keymeta.KeymetaAdminImpl.disableKeyManagement(byte[],
 String)  At KeymetaAdminImpl.java:[line 165] |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.48 ServerAPI=1.48 base: 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7460/6/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hbase/pull/7460 |
   | Optional Tests | dupname asflicense javac spotbugs checkstyle codespell 
detsecrets compile hadoopcheck hbaseanti spotless cc buflint bufcompat 
hbaseprotoc rubocop |
   | uname | Linux 5dbc665d632d 6.8.0-1024-aws #26~22.04.1-Ubuntu SMP Wed Feb 
19 06:54:57 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/hbase-personality.sh |
   | git revision | HBASE-29368-key-management-feature / 
88cdcaeb1877778aa184182817c2ecee739f931e |
   | Default Java | Eclipse Adoptium-17.0.11+9 |
   | Max. process+thread count | 71 (vs. ulimit of 30000) |
   | modules | C: hbase-protocol-shaded hbase-common hbase-client hbase-server 
hbase-thrift hbase-shell U: . |
   | Console output | 
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7460/6/console 
|
   | versions | git=2.34.1 maven=3.9.8 spotbugs=4.7.3 rubocop=1.37.1 |
   | 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