Skip to content

HBASE-29695: Fix typo in RestoreRequest's "Builder.withOvewrite()" me… #7510

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
taklwu merged 1 commit into from
Dec 4, 2025
Merged

HBASE-29695: Fix typo in RestoreRequest's "Builder.withOvewrite()" me… #7510

taklwu merged 1 commit into from
Dec 4, 2025

Conversation

@taklwu
Copy link
Contributor

@taklwu taklwu commented Dec 4, 2025 •
edited
Loading

…thod (#7508)

this is clean cherry-pick from 6d342cc

@kgeisz @taklwu
HBASE-29695: Fix typo in RestoreRequest's "Builder.withOvewrite()" me…
0c69bce 
…thod (apache#7508)

Signed-off-by: Tak Lon (Stephen) Wu <taklwu@apache.org>
Signed-off-by: Andor Molnár <andor@apache.org>
@Apache-HBase
Copy link

Apache-HBase commented Dec 4, 2025

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 0s Docker mode activated.
-1 ❌ docker 0m 26s Docker failed to build run-specific yetus/hbase:tp-18868}.
Subsystem Report/Notes
GITHUB PR #7510
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7510/1/console
versions git=2.17.1
Powered by Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

Apache-HBase commented Dec 4, 2025

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 1m 24s Docker mode activated.
-0 ⚠️ yetus 0m 5s Unprocessed 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 🆗 mvndep 0m 16s Maven dependency ordering for branch
+1 💚 mvninstall 4m 3s branch-2 passed
+1 💚 compile 0m 50s branch-2 passed
+1 💚 javadoc 0m 35s branch-2 passed
+1 💚 shadedjars 6m 24s branch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 16s Maven dependency ordering for patch
+1 💚 mvninstall 3m 7s the patch passed
+1 💚 compile 0m 54s the patch passed
+1 💚 javac 0m 54s the patch passed
+1 💚 javadoc 0m 35s the patch passed
+1 💚 shadedjars 6m 32s patch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚 unit 13m 51s hbase-backup in the patch passed.
+1 💚 unit 1m 16s hbase-it in the patch passed.
41m 20s
Subsystem Report/Notes
Docker ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7510/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR #7510
Optional Tests javac javadoc unit compile shadedjars
uname Linux 215ce364ac23 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 branch-2 / 0c69bce
Default Java Temurin-1.8.0_412-b08
Test Results https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7510/1/testReport/
Max. process+thread count 3443 (vs. ulimit of 30000)
modules C: hbase-backup hbase-it U: .
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7510/1/console
versions git=2.34.1 maven=3.9.8
Powered by Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

Apache-HBase commented Dec 4, 2025

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 1m 2s Docker mode activated.
-0 ⚠️ yetus 0m 4s Unprocessed 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 🆗 mvndep 0m 16s Maven dependency ordering for branch
+1 💚 mvninstall 4m 53s branch-2 passed
+1 💚 compile 0m 58s branch-2 passed
+1 💚 javadoc 0m 37s branch-2 passed
+1 💚 shadedjars 7m 23s branch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 23s Maven dependency ordering for patch
+1 💚 mvninstall 3m 55s the patch passed
+1 💚 compile 0m 53s the patch passed
+1 💚 javac 0m 53s the patch passed
+1 💚 javadoc 0m 29s the patch passed
+1 💚 shadedjars 7m 49s patch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚 unit 12m 52s hbase-backup in the patch passed.
+1 💚 unit 0m 53s hbase-it in the patch passed.
43m 56s
Subsystem Report/Notes
Docker ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7510/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR #7510
Optional Tests javac javadoc unit compile shadedjars
uname Linux 2c82224ca39d 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 branch-2 / 0c69bce
Default Java Eclipse Adoptium-17.0.11+9
Test Results https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7510/1/testReport/
Max. process+thread count 3585 (vs. ulimit of 30000)
modules C: hbase-backup hbase-it U: .
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7510/1/console
versions git=2.34.1 maven=3.9.8
Powered by Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

Apache-HBase commented Dec 4, 2025

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 52s Docker mode activated.
-0 ⚠️ yetus 0m 5s Unprocessed 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 🆗 mvndep 0m 19s Maven dependency ordering for branch
+1 💚 mvninstall 3m 34s branch-2 passed
+1 💚 compile 0m 45s branch-2 passed
+1 💚 javadoc 0m 53s branch-2 passed
+1 💚 shadedjars 10m 18s branch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 23s Maven dependency ordering for patch
+1 💚 mvninstall 4m 40s the patch passed
+1 💚 compile 0m 49s the patch passed
+1 💚 javac 0m 49s the patch passed
+1 💚 javadoc 0m 31s the patch passed
+1 💚 shadedjars 7m 38s patch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚 unit 11m 49s hbase-backup in the patch passed.
+1 💚 unit 0m 55s hbase-it in the patch passed.
44m 52s
Subsystem Report/Notes
Docker ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7510/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR #7510
Optional Tests javac javadoc unit compile shadedjars
uname Linux c648879feae1 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 branch-2 / 0c69bce
Default Java Eclipse Adoptium-11.0.23+9
Test Results https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7510/1/testReport/
Max. process+thread count 3610 (vs. ulimit of 30000)
modules C: hbase-backup hbase-it U: .
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7510/1/console
versions git=2.34.1 maven=3.9.8
Powered by Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

kgeisz
kgeisz approved these changes Dec 4, 2025
View reviewed changes
Copy link
Contributor

@kgeisz kgeisz left a comment

Choose a reason for hiding this comment

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

LGTM

@taklwu
Copy link
Contributor Author

taklwu commented Dec 4, 2025

yetus failed but all the following passed, merging

@taklwu taklwu merged commit fdcc915 into apache:branch-2 Dec 4, 2025
1 check failed
@taklwu taklwu deleted the branch-2 branch December 4, 2025 04:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@kgeisz kgeisz kgeisz approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@taklwu @Apache-HBase @kgeisz