Skip to content

TEZ-4575: Upgrade Apache parent pom version from 23 to 37 - #468

Merged
abstractdog merged 1 commit into
apache:masterfrom
Aggarwal-Raghav:TEZ-4575
Apr 8, 2026
Merged

TEZ-4575: Upgrade Apache parent pom version from 23 to 37#468
abstractdog merged 1 commit into
apache:masterfrom
Aggarwal-Raghav:TEZ-4575

Conversation

@Aggarwal-Raghav

Copy link
Copy Markdown
Contributor

No description provided.

@tez-yetus

Copy link
Copy Markdown

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-468/1/console in case of problems.


######
# Install maven
# Install Maven 3.9.14

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.

Upgrading maven version as apache parent 37 uses maven 3.9 minimum

Comment threadpom.xml
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-source-plugin</artifactId>
<executions>

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.

Removing gpg and sources profiles as they are run in -Papache-release profile present in apace parent 37 pom https://maven.apache.org/pom/asf/

@tez-yetus

This comment was marked as outdated.

@tez-yetus

Copy link
Copy Markdown

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-468/2/console in case of problems.

@Aggarwal-Raghav

Copy link
Copy Markdown
ContributorAuthor

@zabetak , if you have bandwidth can you please help with the review?

@tez-yetus

This comment was marked as outdated.

@Aggarwal-Raghav

Copy link
Copy Markdown
ContributorAuthor

@zabetak , if you have bandwidth can you please help with the review?

Ignore this for now. #470 needs to be addressed first.

@tez-yetus

Copy link
Copy Markdown

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-468/3/console in case of problems.

@tez-yetus

This comment was marked as outdated.

@tez-yetus

Copy link
Copy Markdown

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-468/4/console in case of problems.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec5m 15sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+0 🆗shelldocs0m 0sShelldocs was not available.
+0 🆗xmllint0m 0sxmllint was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+0 🆗mvndep0m 42sMaven dependency ordering for branch
+1 💚mvninstall3m 56smaster passed
+1 💚compile3m 3smaster passed
+1 💚javadoc2m 10smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 10sMaven dependency ordering for patch
+1 💚mvninstall3m 30sthe patch passed
+1 💚codespell1m 5sNo new issues.
+1 💚compile2m 33sthe patch passed
-1 ❌javac0m 24s/results-compile-javac-tez-dag.txttez-dag generated 13 new + 0 unchanged - 0 fixed = 13 total (was 0)
-1 ❌javac0m 18s/results-compile-javac-tez-tests.txttez-tests generated 4 new + 0 unchanged - 0 fixed = 4 total (was 0)
-1 ❌javac1m 3s/results-compile-javac-root.txtroot generated 30 new + 4 unchanged - 0 fixed = 34 total (was 4)
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚hadolint0m 2sNo new issues.
+1 💚shellcheck0m 0sNo new issues.
-1 ❌javadoc0m 52s/results-javadoc-javadoc-root.txtroot generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
_ Other Tests _
+1 💚unit4m 20stez-dag in the patch passed.
+1 💚unit33m 37stez-tests in the patch passed.
+1 💚unit3m 36stez-ext-service-tests in the patch passed.
+1 💚unit2m 30stez-aux-services in the patch passed.
+1 💚unit0m 13sdocs in the patch passed.
+1 💚unit60m 1sroot in the patch passed.
+1 💚asflicense2m 5sThe patch does not generate ASF License warnings.
132m 14s
SubsystemReport/Notes
DockerClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-468/4/artifact/out/Dockerfile
GITHUB PR#468
Optional Testsdupname asflicense codespell detsecrets hadolint shellcheck shelldocs javac javadoc unit xmllint compile
unameLinux fa0be818a771 5.15.0-171-generic #181-Ubuntu SMP Fri Feb 6 22:44:50 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-468/src/.yetus/personality.sh
git revisionmaster / 71cb744
Default JavaUbuntu-21.0.10+7-Ubuntu-124.04
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-468/4/testReport/
Max. process+thread count1423 (vs. ulimit of 5500)
modulesC: tez-dag tez-tests tez-ext-service-tests tez-plugins/tez-aux-services docs . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-468/4/console
versionsgit=2.43.0 maven=3.9.14 hadolint=1.18.0-0-g76eee5c codespell=2.4.1 shellcheck=0.7.1
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@Aggarwal-Raghav

Copy link
Copy Markdown
ContributorAuthor

This is ready for review!

@Aggarwal-Raghav

Copy link
Copy Markdown
ContributorAuthor

CC @abstractdog

@abstractdog

abstractdog commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

changes look good to me, spot-checked some commands:

mvn clean install -DskipTests # OK
mvn clean install javadoc:aggregate -DskipTests -pl tez-api -pl tez-mapreduce -pl tez-runtime-library # OK
mvn dependency:tree #OK
mvn compile spotbugs:spotbugs #OK, htmls generated properly
mvn site -f tez-api #FAILED, not related created TEZ-4702
mvn clean deploy -DskipTests -Papache-release # FAILED: not related to this patch, need to double-check my apache staging credentials

@abstractdog
abstractdog merged commit 2406653 into apache:masterApr 8, 2026
4 checks passed
@Aggarwal-Raghav
Aggarwal-Raghav deleted the TEZ-4575 branch May 1, 2026 05:53
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

@Aggarwal-Raghav@tez-yetus@abstractdog