Skip to content

Backport "HBASE-28865 Implement proper string builder for MoveRegionRequest in ProtobufUtil.getShortTextFormat" to branch-2 - #6362

Merged
stoty merged 5 commits into
apache:branch-2from
PDavid:HBASE-28865-MoveRegionRequest-toString-branch-2
Oct 14, 2024
Merged

Backport "HBASE-28865 Implement proper string builder for MoveRegionRequest in ProtobufUtil.getShortTextFormat" to branch-2#6362
stoty merged 5 commits into
apache:branch-2from
PDavid:HBASE-28865-MoveRegionRequest-toString-branch-2

Conversation

@PDavid

@PDavidPDavid commented Oct 10, 2024•
edited
Loading

Copy link
Copy Markdown
Contributor

This is a backport of #6318 to branch-2.

@Apache-HBase

Apache-HBase commented Oct 10, 2024

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 46sDocker 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 29sbranch-2 passed
+1 💚compile0m 20sbranch-2 passed
+1 💚javadoc0m 19sbranch-2 passed
+1 💚shadedjars4m 56sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+1 💚mvninstall2m 28sthe patch passed
+1 💚compile0m 21sthe patch passed
+1 💚javac0m 21sthe patch passed
+1 💚javadoc0m 18sthe patch passed
+1 💚shadedjars4m 56spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit7m 31shbase-client in the patch passed.
25m 32s
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6362/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR#6362
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 334f6f54b976 5.4.0-195-generic #215-Ubuntu SMP Fri Aug 2 18:28:05 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / 573416d
Default JavaTemurin-1.8.0_412-b08
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6362/1/testReport/
Max. process+thread count354 (vs. ulimit of 30000)
modulesC: hbase-client U: hbase-client
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6362/1/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 10, 2024

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 40sDocker 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 💚mvninstall3m 5sbranch-2 passed
+1 💚compile0m 23sbranch-2 passed
+1 💚javadoc0m 22sbranch-2 passed
+1 💚shadedjars5m 31sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+1 💚mvninstall3m 2sthe patch passed
+1 💚compile0m 24sthe patch passed
+1 💚javac0m 24sthe patch passed
+1 💚javadoc0m 20sthe patch passed
+1 💚shadedjars5m 34spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit7m 36shbase-client in the patch passed.
28m 4s
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6362/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#6362
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 0e1b068ac560 5.4.0-192-generic #212-Ubuntu SMP Fri Jul 5 09:47:39 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / 573416d
Default JavaEclipse Adoptium-11.0.23+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6362/1/testReport/
Max. process+thread count372 (vs. ulimit of 30000)
modulesC: hbase-client U: hbase-client
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6362/1/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 10, 2024

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 43sDocker mode activated.
-0 ⚠️yetus0m 4sUnprocessed 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 9sbranch-2 passed
+1 💚compile0m 23sbranch-2 passed
+1 💚javadoc0m 19sbranch-2 passed
+1 💚shadedjars5m 42sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+1 💚mvninstall2m 56sthe patch passed
+1 💚compile0m 23sthe patch passed
+1 💚javac0m 23sthe patch passed
+1 💚javadoc0m 19sthe patch passed
+1 💚shadedjars5m 43spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit8m 8shbase-client in the patch passed.
28m 53s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6362/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6362
Optional Testsjavac javadoc unit compile shadedjars
unameLinux a912901d1d3e 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 / 573416d
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6362/1/testReport/
Max. process+thread count376 (vs. ulimit of 30000)
modulesC: hbase-client U: hbase-client
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6362/1/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 10, 2024

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 44sDocker 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 12sbranch-2 passed
+1 💚compile0m 54sbranch-2 passed
+1 💚checkstyle0m 21sbranch-2 passed
+1 💚spotbugs0m 56sbranch-2 passed
+1 💚spotless0m 48sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+1 💚mvninstall2m 58sthe patch passed
+1 💚compile0m 53sthe patch passed
+1 💚javac0m 53sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle0m 20shbase-client: The patch generated 0 new + 17 unchanged - 1 fixed = 17 total (was 18)
+1 💚spotbugs1m 1sthe patch passed
+1 💚hadoopcheck15m 54sPatch does not cause any errors with Hadoop 2.10.2 or 3.3.6 3.4.0.
+1 💚spotless0m 46spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 12sThe patch does not generate ASF License warnings.
30m 31s
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6362/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6362
Optional Testsdupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
unameLinux 50669e9995f3 5.4.0-192-generic #212-Ubuntu SMP Fri Jul 5 09:47:39 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / 573416d
Default JavaEclipse Adoptium-11.0.23+9
Max. process+thread count77 (vs. ulimit of 30000)
modulesC: hbase-client U: hbase-client
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6362/1/console
versionsgit=2.34.1 maven=3.9.8 spotbugs=4.7.3
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@PDavid
PDavid marked this pull request as ready for review October 10, 2024 12:49
@PDavidPDavid mentioned this pull request Oct 10, 2024
Merged
@stoty
stoty merged commit 9a95d56 into apache:branch-2Oct 14, 2024
stoty pushed a commit that referenced this pull request Oct 14, 2024
@PDavidDávid Paksy@stoty
HBASE-28865Implement proper string builder for MoveRegionRequest in …
17afc39
…ProtobufUtil.getShortTextFormat (#6362)
Co-authored-by: Dávid Paksy <paksyd@cloudera.com>
Signed-off-by: Istvan Toth <stoty@apache.org>
(cherry picked from commit 9a95d56)
@PDavid
PDavid deleted the HBASE-28865-MoveRegionRequest-toString-branch-2 branch October 15, 2024 07:18
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

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

@PDavid@Apache-HBase@stoty