Skip to content

TEZ-4702: Javadoc generation fails since Java21 - #478

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

TEZ-4702: Javadoc generation fails since Java21#478
abstractdog merged 1 commit into
apache:masterfrom
Aggarwal-Raghav:TEZ-4702

Conversation

@Aggarwal-Raghav

@Aggarwal-RaghavAggarwal-Raghav commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

This is blocked on TEZ-4688, until Hadoop is upgraded to 3.5.0 in tez.
HADOOP-19402, HADOOP-19785 and other patches has fixed IncludePublicAnnotationsStandardDoclet class to support JDK17+ syntax

Commands to verify:

mvn clean install javadoc:aggregate -DskipTests -pl tez-api -pl tez-mapreduce -pl tez-runtime-library -Dhadoop.version=3.5.0
mvn site -f tez-api -Dhadoop.version=3.5.0

@Aggarwal-Raghav

Copy link
Copy Markdown
ContributorAuthor

mvn site -f tez-api -Dhadoop.version=3.5.0 is working
Screenshot 2026-04-11 at 12 09 28 AM

@Aggarwal-Raghav

Copy link
Copy Markdown
ContributorAuthor

There is a scope to get rid of tez-tools/tez-javadoc-tools/src/main/java-8-16 and rename java17 and make it standard

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec5m 17sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo case conflicting files found.
+0 🆗detsecrets0m 0sdetect-secrets 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 _
+1 💚mvninstall2m 55smaster passed
+1 💚compile1m 2smaster passed
+1 💚javadoc0m 59smaster passed
_ Patch Compile Tests _
+1 💚mvninstall2m 0sthe patch passed
+1 💚codespell0m 48sNo new issues.
+1 💚compile1m 4sthe patch passed
+1 💚javac1m 4sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚javadoc0m 54sthe patch passed
_ Other Tests _
+1 💚unit59m 58sroot in the patch passed.
+1 💚asflicense0m 34sThe patch does not generate ASF License warnings.
76m 27s
SubsystemReport/Notes
DockerClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-478/1/artifact/out/Dockerfile
GITHUB PR#478
Optional Testsdupname asflicense javac javadoc unit codespell detsecrets xmllint compile
unameLinux 94e32068ae22 5.15.0-141-generic #151-Ubuntu SMP Sun May 18 21:35:19 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-478/src/.yetus/personality.sh
git revisionmaster / c4196e7
Default JavaUbuntu-21.0.10+7-Ubuntu-124.04
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-478/1/testReport/
Max. process+thread count1300 (vs. ulimit of 5500)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-478/1/console
versionsgit=2.43.0 maven=3.9.14 codespell=2.4.1
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@abstractdog

Copy link
Copy Markdown
Contributor

patch seems to work with local testing, so we don't even have to wait for the hadoop 3.5.0 upgrade to happen, LGTM

@abstractdog
abstractdog merged commit f2436ae into apache:masterApr 13, 2026
4 checks passed
@Aggarwal-Raghav
Aggarwal-Raghav deleted the TEZ-4702 branch April 28, 2026 19:25
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