Skip to content

Revert "HBASE-29225 Add module for Jetty 12 with EE8 to hbase-thirdpa… - #140

Merged
NihalJain merged 1 commit into
apache:masterfrom
NihalJain:master
May 26, 2025
Merged

Revert "HBASE-29225 Add module for Jetty 12 with EE8 to hbase-thirdpa…#140
NihalJain merged 1 commit into
apache:masterfrom
NihalJain:master

Conversation

@NihalJain

Copy link
Copy Markdown
Contributor

…rty (#131)"

This reverts commit 6016e75.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 25sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-0 ⚠️test4tests0m 0sThe 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 🆗mvndep0m 30sMaven dependency ordering for branch
-1 ❌mvninstall0m 17sroot in master failed.
-1 ❌compile0m 18sroot in master failed.
-1 ❌compile0m 17shbase-shaded-jetty-12-plus-core in master failed.
-1 ❌compile0m 17shbase-shaded-jetty-12-plus-ee8 in master failed.
-1 ❌javadoc0m 18sroot in master failed.
-1 ❌javadoc0m 17shbase-shaded-jetty-12-plus-core in master failed.
-1 ❌javadoc0m 17shbase-shaded-jetty-12-plus-ee8 in master failed.
_ Patch Compile Tests _
-1 ❌mvninstall0m 17sroot in the patch failed.
-1 ❌compile0m 18sroot in the patch failed.
-1 ❌javac0m 18sroot in the patch failed.
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 1sThe patch has no ill-formed XML file.
-1 ❌javadoc0m 17sroot in the patch failed.
_ Other Tests _
-1 ❌unit0m 17sroot in the patch failed.
+0 🆗asflicense0m 17sASF License check generated no output?
6m 20s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-140/1/artifact/yetus-precommit-check/output/Dockerfile
GITHUB PR#140
Optional Testsdupname asflicense javac javadoc unit xml compile
unameLinux d4ca5173e71a 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 toolmaven
git revisionmaster / 223f4c6
Default JavaTemurin-1.8.0_452-b09
mvninstallhttps://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-140/1/artifact/yetus-precommit-check/output/branch-mvninstall-root.txt
compilehttps://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-140/1/artifact/yetus-precommit-check/output/branch-compile-root.txt
compilehttps://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-140/1/artifact/yetus-precommit-check/output/branch-compile-hbase-shaded-jetty-12-plus-core.txt
compilehttps://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-140/1/artifact/yetus-precommit-check/output/branch-compile-hbase-shaded-jetty-12-plus-ee8.txt
javadochttps://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-140/1/artifact/yetus-precommit-check/output/branch-javadoc-root.txt
javadochttps://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-140/1/artifact/yetus-precommit-check/output/branch-javadoc-hbase-shaded-jetty-12-plus-core.txt
javadochttps://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-140/1/artifact/yetus-precommit-check/output/branch-javadoc-hbase-shaded-jetty-12-plus-ee8.txt
mvninstallhttps://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-140/1/artifact/yetus-precommit-check/output/patch-mvninstall-root.txt
compilehttps://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-140/1/artifact/yetus-precommit-check/output/patch-compile-root.txt
javachttps://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-140/1/artifact/yetus-precommit-check/output/patch-compile-root.txt
javadochttps://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-140/1/artifact/yetus-precommit-check/output/patch-javadoc-root.txt
unithttps://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-140/1/artifact/yetus-precommit-check/output/patch-unit-root.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-140/1/testReport/
Max. process+thread count22 (vs. ulimit of 1000)
modulesC: . U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-140/1/console
versionsgit=2.43.0 maven=3.9.9
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@NihalJain

NihalJain commented May 26, 2025

Copy link
Copy Markdown
ContributorAuthor

Our CI seems to be broken, since a simple revert and is approved going ahead with merge. Have built code locally to ensure no failure.

[INFO] Reactor Summary for Apache HBase Third-Party Libs 4.1.11-SNAPSHOT:
[INFO]
[INFO] Apache HBase Third-Party Libs ...................... SUCCESS [ 4.155 s]
[INFO] Apache HBase Patched and Relocated (Shaded) Protobuf SUCCESS [ 29.529 s]
[INFO] Apache HBase Relocated (Shaded) Netty Libs ......... SUCCESS [ 13.403 s]
[INFO] Apache HBase Relocated (Shaded) netty-tcnative Libs SUCCESS [ 5.339 s]
[INFO] Apache HBase Relocated (Shaded) GSON Libs .......... SUCCESS [ 0.798 s]
[INFO] Apache HBase Relocated (Shaded) Third-party Miscellaneous Libs SUCCESS [ 9.238 s]
[INFO] Apache HBase Relocated (Shaded) Jetty Libs ......... SUCCESS [ 6.394 s]
[INFO] Apache HBase Relocated (Shaded) Jersey Libs ........ SUCCESS [ 26.180 s]
[INFO] Apache HBase Relocated (Shaded) jackson-jaxrs-json-provider SUCCESS [ 1.459 s]
[INFO] Apache HBase Drop-in noop HTrace replacement ....... SUCCESS [01:13 min]
[INFO] Apache HBase Unsafe Wrapper ........................ SUCCESS [ 1.435 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 02:52 min
[INFO] Finished at: 2025-05-26T19:47:09Z
[INFO] ------------------------------------------------------------------------

@NihalJainNihalJain reopened this May 26, 2025
@NihalJain
NihalJain merged commit 00249ba into apache:masterMay 26, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@NihalJain@Apache-HBase@Apache9