Apache-HBase commented on PR #142: URL: https://github.com/apache/hbase-thirdparty/pull/142#issuecomment-3090955690
:broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |:----:|----------:|--------:|:--------| | +0 :ok: | reexec | 0m 31s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +0 :ok: | shelldocs | 0m 0s | Shelldocs was not available. | | +0 :ok: | hadolint | 0m 0s | hadolint was not available. | | +0 :ok: | markdownlint | 0m 0s | markdownlint was not available. | | +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. | | -0 :warning: | test4tests | 0m 0s | The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. | ||| _ master Compile Tests _ | | +0 :ok: | mvndep | 0m 4s | Maven dependency ordering for branch | | -1 :x: | mvninstall | 0m 36s | root in master failed. | | -1 :x: | compile | 0m 6s | hbase-unsafe in master failed. | | -1 :x: | compile | 0m 14s | root in master failed. | | +1 :green_heart: | javadoc | 0m 17s | master passed | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 6s | Maven dependency ordering for patch | | +1 :green_heart: | mvninstall | 1m 16s | the patch passed | | +1 :green_heart: | compile | 0m 44s | the patch passed | | -1 :x: | javac | 0m 11s | hbase-shaded-protobuf generated 19 new + 0 unchanged - 21 fixed = 19 total (was 21) | | -1 :x: | javac | 0m 6s | hbase-unsafe generated 8 new + 0 unchanged - 18 fixed = 8 total (was 18) | | -1 :x: | javac | 0m 16s | root generated 27 new + 0 unchanged - 39 fixed = 27 total (was 39) | | +1 :green_heart: | shellcheck | 0m 0s | There were no new shellcheck issues. | | -1 :x: | 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 :x: | xml | 0m 0s | The patch has 6 ill-formed XML file(s). | | +1 :green_heart: | javadoc | 0m 26s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | unit | 0m 11s | hbase-shaded-protobuf in the patch passed. | | +1 :green_heart: | unit | 0m 5s | hbase-shaded-jetty-12-plus-core in the patch passed. | | +1 :green_heart: | unit | 0m 5s | hbase-shaded-jetty-12-plus-ee8 in the patch passed. | | +1 :green_heart: | unit | 0m 6s | hbase-unsafe in the patch passed. | | +1 :green_heart: | unit | 0m 45s | root in the patch passed. | | +1 :green_heart: | asflicense | 0m 24s | The patch does not generate ASF License warnings. | | | | 6m 26s | | | Reason | Tests | |-------:|:------| | XML | Parsing Error(s): | | | dev-support/toolchains-jenkins.xml | | | hbase-shaded-jetty-12-plus-core/pom.xml | | | hbase-shaded-jetty-12-plus-ee8/pom.xml | | | hbase-shaded-protobuf/pom.xml | | | hbase-unsafe/pom.xml | | | pom.xml | | Subsystem | Report/Notes | |----------:|:-------------| | Docker | ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-142/50/artifact/yetus-precommit-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase-thirdparty/pull/142 | | Optional Tests | dupname asflicense shellcheck shelldocs hadolint xml javac javadoc unit compile markdownlint | | uname | Linux b3742e966585 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 | | git revision | master / 1e93a44 | | Default Java | Eclipse Adoptium-17.0.11+9 | | mvninstall | https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-142/50/artifact/yetus-precommit-check/output/branch-mvninstall-root.txt | | compile | https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-142/50/artifact/yetus-precommit-check/output/branch-compile-hbase-unsafe.txt | | compile | https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-142/50/artifact/yetus-precommit-check/output/branch-compile-root.txt | | javac | https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-142/50/artifact/yetus-precommit-check/output/diff-compile-javac-hbase-shaded-protobuf.txt | | javac | https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-142/50/artifact/yetus-precommit-check/output/diff-compile-javac-hbase-unsafe.txt | | javac | https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-142/50/artifact/yetus-precommit-check/output/diff-compile-javac-root.txt | | whitespace | https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-142/50/artifact/yetus-precommit-check/output/whitespace-eol.txt | | xml | https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-142/50/artifact/yetus-precommit-check/output/xml.txt | | Test Results | https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-142/50/testReport/ | | Max. process+thread count | 416 (vs. ulimit of 1000) | | modules | C: hbase-shaded-protobuf hbase-shaded-jetty-12-plus-core hbase-shaded-jetty-12-plus-ee8 hbase-unsafe . U: . | | Console output | https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-142/50/console | | versions | git=2.34.1 maven=file shellcheck=0.8.0 | | 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