Skip to content

HBASE-29614 Remove static final field modification in tests around Un… - #7347

Merged
Apache9 merged 2 commits into
apache:branch-2from
Apache9:HBASE-29614-branch-2
Oct 8, 2025
Merged

HBASE-29614 Remove static final field modification in tests around Un…#7347
Apache9 merged 2 commits into
apache:branch-2from
Apache9:HBASE-29614-branch-2

Conversation

@Apache9

@Apache9Apache9 commented Sep 29, 2025

Copy link
Copy Markdown
Contributor

…safe (#7337)

(cherry picked from commit e0cec31)

@Apache9
HBASE-29614 Remove static final field modification in tests around Un…
e2be378
…safe (apache#7337)
Signed-off-by: Peng Lu <lupeng@apache.org>
(cherry picked from commit e0cec31)
@Apache9Apache9 self-assigned this Sep 29, 2025
@Apache9Apache9 added the backport This PR is a back port of some issue or issues already committed to master label Sep 29, 2025
@Apache-HBase

This comment has been minimized.

Sign in to view
@Apache-HBase

This comment has been minimized.

Sign in to view
@Apache-HBase

This comment has been minimized.

Sign in to view
@Apache-HBase

This comment has been minimized.

Sign in to view
@Apache-HBase

This comment has been minimized.

Sign in to view
@Apache-HBase

This comment has been minimized.

Sign in to view
@Apache-HBase

This comment has been minimized.

Sign in to view
@Apache-HBase

This comment has been minimized.

Sign in to view
@Apache-HBase

Apache-HBase commented Oct 7, 2025

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 50sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo 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 _
+0 🆗mvndep0m 21sMaven dependency ordering for branch
+1 💚mvninstall3m 34sbranch-2 passed
+1 💚compile3m 47sbranch-2 passed
+1 💚checkstyle0m 55sbranch-2 passed
+1 💚spotbugs2m 10sbranch-2 passed
+1 💚spotless0m 51sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+0 🆗mvndep0m 21sMaven dependency ordering for patch
+1 💚mvninstall3m 9sthe patch passed
+1 💚compile3m 39sthe patch passed
+1 💚javac0m 34shbase-common generated 0 new + 102 unchanged - 1 fixed = 102 total (was 103)
-0 ⚠️javac3m 5s/results-compile-javac-hbase-server.txthbase-server generated 1 new + 194 unchanged - 1 fixed = 195 total (was 195)
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle0m 14sThe patch passed checkstyle in hbase-common
+1 💚checkstyle0m 38shbase-server: The patch generated 0 new + 2 unchanged - 4 fixed = 2 total (was 6)
+1 💚xmllint0m 0sNo new issues.
+1 💚spotbugs2m 21sthe patch passed
+1 💚hadoopcheck17m 19sPatch does not cause any errors with Hadoop 2.10.2 or 3.3.6 3.4.1.
+1 💚spotless0m 45spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 19sThe patch does not generate ASF License warnings.
43m 19s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7347/3/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#7347
Optional Testsdupname asflicense javac codespell detsecrets xmllint hadoopcheck spotless compile spotbugs checkstyle hbaseanti
unameLinux 38bd6afb6e30 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 / caa6fe7
Default JavaEclipse Adoptium-11.0.23+9
Max. process+thread count80 (vs. ulimit of 30000)
modulesC: hbase-common hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7347/3/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.

@Apache-HBase

Apache-HBase commented Oct 7, 2025

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 _
+0 🆗mvndep0m 32sMaven dependency ordering for branch
+1 💚mvninstall2m 52sbranch-2 passed
+1 💚compile0m 58sbranch-2 passed
+1 💚javadoc0m 38sbranch-2 passed
+1 💚shadedjars5m 36sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 14sMaven dependency ordering for patch
+1 💚mvninstall2m 29sthe patch passed
+1 💚compile0m 59sthe patch passed
+1 💚javac0m 59sthe patch passed
+1 💚javadoc0m 38sthe patch passed
+1 💚shadedjars5m 32spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit2m 3shbase-common in the patch passed.
+1 💚unit213m 43shbase-server in the patch passed.
241m 58s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7347/3/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR#7347
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 25ed803609d9 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 / caa6fe7
Default JavaTemurin-1.8.0_412-b08
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7347/3/testReport/
Max. process+thread count3226 (vs. ulimit of 30000)
modulesC: hbase-common hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7347/3/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

Apache-HBase commented Oct 7, 2025

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 46sDocker mode activated.
-0 ⚠️yetus0m 7sUnprocessed 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 _
+0 🆗mvndep0m 20sMaven dependency ordering for branch
+1 💚mvninstall3m 31sbranch-2 passed
+1 💚compile1m 11sbranch-2 passed
+1 💚javadoc0m 45sbranch-2 passed
+1 💚shadedjars6m 46sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 21sMaven dependency ordering for patch
+1 💚mvninstall3m 7sthe patch passed
+1 💚compile1m 11sthe patch passed
+1 💚javac1m 11sthe patch passed
+1 💚javadoc0m 43sthe patch passed
+1 💚shadedjars6m 43spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit2m 39shbase-common in the patch passed.
+1 💚unit222m 54shbase-server in the patch passed.
255m 33s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7347/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#7347
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 0dc5b5cac9eb 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 / caa6fe7
Default JavaEclipse Adoptium-11.0.23+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7347/3/testReport/
Max. process+thread count3376 (vs. ulimit of 30000)
modulesC: hbase-common hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7347/3/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

Apache-HBase commented Oct 7, 2025

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 58sDocker mode activated.
-0 ⚠️yetus0m 8sUnprocessed 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 _
+0 🆗mvndep0m 21sMaven dependency ordering for branch
+1 💚mvninstall5m 29sbranch-2 passed
+1 💚compile2m 3sbranch-2 passed
+1 💚javadoc1m 9sbranch-2 passed
+1 💚shadedjars9m 54sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 34sMaven dependency ordering for patch
+1 💚mvninstall4m 55sthe patch passed
+1 💚compile2m 5sthe patch passed
+1 💚javac2m 5sthe patch passed
+1 💚javadoc1m 10sthe patch passed
+1 💚shadedjars9m 27spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit3m 39shbase-common in the patch passed.
-1 ❌unit254m 19s/patch-unit-hbase-server.txthbase-server in the patch failed.
301m 22s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7347/3/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#7347
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 633df934396c 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 / caa6fe7
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7347/3/testReport/
Max. process+thread count3358 (vs. ulimit of 30000)
modulesC: hbase-common hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7347/3/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache9
Apache9 merged commit 101c674 into apache:branch-2Oct 8, 2025
1 check failed
Apache9 added a commit that referenced this pull request Oct 8, 2025
@Apache9
HBASE-29614Remove static final field modification in tests around Un…
d46a577
…safe (#7337) (#7347)
Signed-off-by: Peng Lu <lupeng@apache.org>
(cherry picked from commit e0cec31)
Apache9 added a commit that referenced this pull request Oct 8, 2025
@Apache9
HBASE-29614Remove static final field modification in tests around Un…
6b887db
…safe (#7337) (#7347)
Signed-off-by: Peng Lu <lupeng@apache.org>
(cherry picked from commit e0cec31)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@Apache9Apache9

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Apache9@Apache-HBase