Apache-HBase commented on PR #6044: URL: https://github.com/apache/hbase/pull/6044#issuecomment-2208685245
:broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |:----:|----------:|--------:|:--------:|:-------:| | +0 :ok: | reexec | 0m 36s | | 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 10s | | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 2m 57s | | master passed | | +1 :green_heart: | compile | 6m 18s | | master passed | | +1 :green_heart: | checkstyle | 1m 57s | | master passed | | +1 :green_heart: | spotbugs | 5m 5s | | master passed | | +1 :green_heart: | spotless | 0m 42s | | branch has no errors when running spotless:check. | |||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 10s | | Maven dependency ordering for patch | | +1 :green_heart: | mvninstall | 2m 42s | | the patch passed | | +1 :green_heart: | compile | 6m 14s | | the patch passed | | -0 :warning: | javac | 0m 37s | [/results-compile-javac-hbase-common.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6044/2/artifact/yetus-general-check/output/results-compile-javac-hbase-common.txt) | hbase-common generated 1 new + 65 unchanged - 0 fixed = 66 total (was 65) | | +1 :green_heart: | blanks | 0m 0s | | The patch has no blanks issues. | | +1 :green_heart: | checkstyle | 0m 12s | | hbase-common: The patch generated 0 new + 17 unchanged - 11 fixed = 17 total (was 28) | | +1 :green_heart: | checkstyle | 0m 15s | | hbase-client: The patch generated 0 new + 26 unchanged - 1 fixed = 26 total (was 27) | | -0 :warning: | checkstyle | 0m 34s | [/results-checkstyle-hbase-server.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6044/2/artifact/yetus-general-check/output/results-checkstyle-hbase-server.txt) | hbase-server: The patch generated 2 new + 71 unchanged - 4 fixed = 73 total (was 75) | | +1 :green_heart: | checkstyle | 0m 11s | | hbase-mapreduce: The patch generated 0 new + 18 unchanged - 5 fixed = 18 total (was 23) | | +1 :green_heart: | checkstyle | 0m 28s | | The patch passed checkstyle in hbase-thrift | | +1 :green_heart: | checkstyle | 0m 8s | | The patch passed checkstyle in hbase-endpoint | | +1 :green_heart: | checkstyle | 0m 8s | | The patch passed checkstyle in hbase-backup | | -1 :x: | spotbugs | 1m 34s | [/new-spotbugs-hbase-server.html](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6044/2/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 | 5m 16s | | Patch does not cause any errors with Hadoop 3.3.6. | | +1 :green_heart: | spotless | 0m 40s | | patch has no errors when running spotless:check. | |||| _ Other Tests _ | | +1 :green_heart: | asflicense | 0m 55s | | The patch does not generate ASF License warnings. | | | | 49m 33s | | | | Reason | Tests | |-------:|:------| | SpotBugs | module:hbase-server | | | instanceof will always return true for all non-null values in org.apache.hadoop.hbase.security.visibility.VisibilityController.preBatchMutate(ObserverContext, MiniBatchOperationInProgress), since all org.apache.hadoop.hbase.ExtendedCell are instances of org.apache.hadoop.hbase.ExtendedCell At VisibilityController.java:for all non-null values in org.apache.hadoop.hbase.security.visibility.VisibilityController.preBatchMutate(ObserverContext, MiniBatchOperationInProgress), since all org.apache.hadoop.hbase.ExtendedCell are instances of org.apache.hadoop.hbase.ExtendedCell At VisibilityController.java:[line 306] | | Subsystem | Report/Notes | |----------:|:-------------| | Docker | ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6044/2/artifact/yetus-general-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/6044 | | Optional Tests | dupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless | | uname | Linux 0c55166fb5cc 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 / e7d079152a579b32e4cc9109e49751e702c3349a | | Default Java | Eclipse Adoptium-17.0.11+9 | | Max. process+thread count | 85 (vs. ulimit of 30000) | | modules | C: hbase-common hbase-client hbase-server hbase-mapreduce hbase-thrift hbase-endpoint hbase-backup U: . | | Console output | https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6044/2/console | | versions | git=2.34.1 maven=3.9.8 spotbugs=4.7.3 | | 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]
