Skip to content

HBASE-29844 Upgrade hbase-metrics to use junit5 - #7659

Merged
PDavid merged 1 commit into
apache:branch-2from
liuxiaocs7:HBASE-29844-branch-2
Jan 28, 2026
Merged

HBASE-29844 Upgrade hbase-metrics to use junit5#7659
PDavid merged 1 commit into
apache:branch-2from
liuxiaocs7:HBASE-29844-branch-2

Conversation

@liuxiaocs7

@liuxiaocs7liuxiaocs7 commented Jan 22, 2026

Copy link
Copy Markdown
Member

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@PDavidPDavid added the backport This PR is a back port of some issue or issues already committed to master label Jan 22, 2026

@PDavidPDavid left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this. 👍

@liuxiaocs7

Copy link
Copy Markdown
MemberAuthor

Hi, @ndimiduk, Yetus General Check seems fails on branch-2, could you help take a look when free, thanks! (https://github.com/apache/hbase/actions/runs/21238548792/job/61111374232?pr=7659)

maven install: branch-2
============================================================================
============================================================================
maven install: branch-2
============================================================================
============================================================================
[Thu Jan 22 06:32:31 AM UTC 2026 INFO]: Personality: branch mvninstall
[Thu Jan 22 06:32:31 AM UTC 2026 DEBUG]: Personality: clear queue
[Thu Jan 22 06:32:31 AM UTC 2026 DEBUG]: Personality: enqueue . --threads=4 -Djava.io.tmpdir=/home/runner/work/hbase/hbase/src/target -DHBasePatchProcess
[Thu Jan 22 06:32:31 AM UTC 2026 DEBUG]: MultiJDK not configured.
cd /home/runner/work/hbase/hbase/src
/opt/maven/bin/mvn --batch-mode -Dmaven.repo.local=/home/runner/yetus-m2/hbase-branch-2-patch-112 --threads=4 -Djava.io.tmpdir=/home/runner/work/hbase/hbase/src/target -DHBasePatchProcess -fae clean install -DskipTests=true -Dmaven.javadoc.skip=true -Dcheckstyle.skip=true -Dfindbugs.skip=true -Dspotbugs.skip=true > /home/runner/work/hbase/hbase/yetus-general-check/output/branch-mvninstall-root.txt 2>&1
[Thu Jan 22 06:32:45 AM UTC 2026 DEBUG]: Checking for unreaped processes:
Elapsed: 0m 16s
[Thu Jan 22 06:32:47 AM UTC 2026 DEBUG]: MultiJDK not configured.
root in branch-2 failed.

@ndimiduk

Copy link
Copy Markdown
Member

Looks like I don't have JDK switching tied to hadoop version switching for branch-2. HBASE-29852.

Thanks @liuxiaocs7 .

@ndimiduk

Copy link
Copy Markdown
Member

#7672

@liuxiaocs7

Copy link
Copy Markdown
MemberAuthor

Looks like I don't have JDK switching tied to hadoop version switching for branch-2. HBASE-29852.

Thanks @liuxiaocs7 .

Hi, @ndimiduk, got it, thanks for your quick feedback!

Signed-off-by: Tak Lon (Stephen) Wu <taklwu@apache.org>
@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 48sDocker mode activated.
-0 ⚠️yetus0m 5sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+1 💚mvninstall2m 52sbranch-2 passed
+1 💚compile0m 11sbranch-2 passed
+1 💚javadoc0m 12sbranch-2 passed
+1 💚shadedjars5m 13sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+1 💚mvninstall2m 28sthe patch passed
+1 💚compile0m 11sthe patch passed
+1 💚javac0m 11sthe patch passed
+1 💚javadoc0m 10sthe patch passed
+1 💚shadedjars5m 10spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit0m 19shbase-metrics in the patch passed.
19m 26s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7659/2/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR#7659
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 7fcdfbfc0aa4 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
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / 667a0d9
Default JavaTemurin-1.8.0_412-b08
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7659/2/testReport/
Max. process+thread count172 (vs. ulimit of 30000)
modulesC: hbase-metrics U: hbase-metrics
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7659/2/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 46sDocker mode activated.
-0 ⚠️yetus0m 6sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+1 💚mvninstall3m 38sbranch-2 passed
+1 💚compile0m 12sbranch-2 passed
+1 💚javadoc0m 13sbranch-2 passed
+1 💚shadedjars6m 26sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+1 💚mvninstall3m 10sthe patch passed
+1 💚compile0m 12sthe patch passed
+1 💚javac0m 12sthe patch passed
+1 💚javadoc0m 11sthe patch passed
+1 💚shadedjars6m 20spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit0m 22shbase-metrics in the patch passed.
23m 50s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7659/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#7659
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 8a882a75b47f 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
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / 667a0d9
Default JavaEclipse Adoptium-11.0.23+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7659/2/testReport/
Max. process+thread count196 (vs. ulimit of 30000)
modulesC: hbase-metrics U: hbase-metrics
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7659/2/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec1m 5sDocker mode activated.
-0 ⚠️yetus0m 6sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+1 💚mvninstall4m 41sbranch-2 passed
+1 💚compile0m 16sbranch-2 passed
+1 💚javadoc0m 17sbranch-2 passed
+1 💚shadedjars8m 15sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+1 💚mvninstall5m 1sthe patch passed
+1 💚compile0m 21sthe patch passed
+1 💚javac0m 21sthe patch passed
+1 💚javadoc0m 21sthe patch passed
+1 💚shadedjars8m 57spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit0m 34shbase-metrics in the patch passed.
31m 12s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7659/2/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#7659
Optional Testsjavac javadoc unit shadedjars compile
unameLinux d76aabef5f45 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
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / 667a0d9
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7659/2/testReport/
Max. process+thread count212 (vs. ulimit of 30000)
modulesC: hbase-metrics U: hbase-metrics
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7659/2/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 48sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 0scodespell was not available.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
_ branch-2 Compile Tests _
+1 💚mvninstall3m 32sbranch-2 passed
+1 💚compile0m 17sbranch-2 passed
+1 💚checkstyle0m 9sbranch-2 passed
+1 💚spotbugs0m 19sbranch-2 passed
+1 💚spotless0m 51sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+1 💚mvninstall3m 12sthe patch passed
+1 💚compile0m 15sthe patch passed
+1 💚javac0m 15sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle0m 8sthe patch passed
+1 💚xmllint0m 0sNo new issues.
+1 💚spotbugs0m 25sthe patch passed
+1 💚hadoopcheck17m 31sPatch does not cause any errors with Hadoop 2.10.2 or 3.3.6 3.4.1.
+1 💚spotless0m 46spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 11sThe patch does not generate ASF License warnings.
31m 46s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7659/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#7659
Optional Testsdupname asflicense javac codespell detsecrets xmllint hadoopcheck spotless compile spotbugs checkstyle hbaseanti
unameLinux 66868f9c3f25 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
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / 667a0d9
Default JavaEclipse Adoptium-11.0.23+9
Max. process+thread count79 (vs. ulimit of 30000)
modulesC: hbase-metrics U: hbase-metrics
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7659/2/console
versionsgit=2.34.1 maven=3.9.8 spotbugs=4.7.3 xmllint=20913
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@liuxiaocs7

liuxiaocs7 commented Jan 28, 2026

Copy link
Copy Markdown
MemberAuthor

Hi, @PDavid, thanks for reviewing, @taklwu, could you help take a look when free, thanks!

@PDavid
PDavid merged commit 095c7db into apache:branch-2Jan 28, 2026
2 checks passed
Apache9 pushed a commit that referenced this pull request Feb 15, 2026
Signed-off-by: Tak Lon (Stephen) Wu <taklwu@apache.org>
Signed-off-by: Dávid Paksy <paksyd@apache.org>
(cherry picked from commit 095c7db)
Apache9 pushed a commit that referenced this pull request Feb 15, 2026
Signed-off-by: Tak Lon (Stephen) Wu <taklwu@apache.org>
Signed-off-by: Dávid Paksy <paksyd@apache.org>
(cherry picked from commit 095c7db)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backportThis PR is a back port of some issue or issues already committed to master

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@liuxiaocs7@Apache-HBase@ndimiduk@PDavid