Skip to content

HBASE-28939 Change the default Hadoop 3 version to 3.4.1 - #6400

Merged
stoty merged 5 commits into
apache:masterfrom
stoty:HBASE-28939
Oct 27, 2024
Merged

HBASE-28939 Change the default Hadoop 3 version to 3.4.1#6400
stoty merged 5 commits into
apache:masterfrom
stoty:HBASE-28939

Conversation

@stoty

Copy link
Copy Markdown
Contributor

No description provided.

@stoty

Copy link
Copy Markdown
ContributorAuthor

We are already running backwards compatibility tests for 3.3.5, 3.3.6 and 3.4.0, and we run the full test suite for the default 3.4.1 version, so IMO no new tests are needed.

@Apache-HBase

This comment has been minimized.

Comment threaddev-support/Jenkinsfile Outdated
@@ -60,7 +60,7 @@ pipeline {
ASF_NIGHTLIES_BASE = "${ASF_NIGHTLIES_BASE_ORI.replaceAll(' ', '%20')}"
// These are dependent on the branch
HADOOP3_VERSIONS = "3.3.5,3.3.6,3.4.0"

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.

should add 3.4.1 to this line too?

@stotystotyOct 25, 2024

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Yes we do, otherwise the caching won't work.

The problem is that backward compatibility test will run for 16 hours now.

We should probably exclude the default version from the backwards compatibility tests.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Added

@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.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 43sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 0scodespell was not available.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+0 🆗shelldocs0m 0sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for branch
+1 💚mvninstall2m 52smaster passed
+1 💚compile8m 3smaster passed
+1 💚spotless0m 41sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for patch
+1 💚mvninstall2m 53sthe patch passed
+1 💚compile7m 56sthe patch passed
+1 💚javac7m 56sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚shellcheck0m 0sNo new issues.
+1 💚xmllint0m 0sNo new issues.
+1 💚hadoopcheck10m 6sPatch does not cause any errors with Hadoop 3.3.6 3.4.0.
+1 💚spotless0m 44spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 37sThe patch does not generate ASF License warnings.
41m 30s
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6400/4/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6400
Optional Testsdupname asflicense codespell detsecrets shellcheck shelldocs spotless javac xmllint hadoopcheck compile
unameLinux 33410260bc66 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 revisionmaster / 5dfdcd1
Default JavaEclipse Adoptium-17.0.11+9
Max. process+thread count193 (vs. ulimit of 30000)
modulesC: hbase-resource-bundle hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants . U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6400/4/console
versionsgit=2.34.1 maven=3.9.8 shellcheck=0.8.0 xmllint=20913
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 41sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 17sMaven dependency ordering for branch
+1 💚mvninstall2m 52smaster passed
+1 💚compile2m 9smaster passed
+1 💚javadoc2m 31smaster passed
+1 💚shadedjars5m 5sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for patch
+1 💚mvninstall2m 51sthe patch passed
+1 💚compile2m 8sthe patch passed
+1 💚javac2m 8sthe patch passed
+1 💚javadoc2m 32sthe patch passed
+1 💚shadedjars5m 20spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit302m 56sroot in the patch passed.
336m 55s
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6400/4/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6400
Optional Testsjavac javadoc unit shadedjars compile
unameLinux bdf6cbbb3634 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 revisionmaster / 5dfdcd1
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6400/4/testReport/
Max. process+thread count9269 (vs. ulimit of 30000)
modulesC: hbase-resource-bundle hbase-shaded/hbase-shaded-check-invariants hbase-shaded/hbase-shaded-with-hadoop-check-invariants . U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6400/4/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@stoty
stoty requested review from Apache9 and ndimidukOctober 27, 2024 06:53
@stoty
stoty merged commit 5f7952a into apache:masterOct 27, 2024
@stoty

Copy link
Copy Markdown
ContributorAuthor

Thank you.
I will start the backports once we had a nightly test run without obvious issues.

stoty added a commit to stoty/hbase that referenced this pull request Oct 28, 2024
stoty added a commit to stoty/hbase that referenced this pull request Oct 28, 2024
stoty added a commit that referenced this pull request Oct 30, 2024
Signed-off-by: Duo Zhang <zhangduo@apache.org>
(cherry picked from commit 5f7952a)
stoty added a commit to stoty/hbase that referenced this pull request Oct 30, 2024
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

@stoty@Apache-HBase@Apache9