Skip to content

TEZ-4713: Sync 3rd party dependency with Hadoop-3.5.0 - #490

Merged
abstractdog merged 1 commit into
apache:masterfrom
Aggarwal-Raghav:TEZ-4713
Jun 22, 2026
Merged

TEZ-4713: Sync 3rd party dependency with Hadoop-3.5.0#490
abstractdog merged 1 commit into
apache:masterfrom
Aggarwal-Raghav:TEZ-4713

Conversation

@Aggarwal-Raghav

@Aggarwal-RaghavAggarwal-Raghav commented Apr 27, 2026

Copy link
Copy Markdown
Contributor

Comment threadpom.xml
<!--dependency versions in alphabetical order-->
<asynchttpclient.version>2.12.4</asynchttpclient.version>
<bouncycastle.version>1.79</bouncycastle.version>
<bouncycastle.version>1.82</bouncycastle.version>

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.

to be in sync with hadoop-3.5.0

Comment threadpom.xml
<plexus-velocity.version>2.3.0</plexus-velocity.version>
<frontend-maven-plugin.version>1.15.0</frontend-maven-plugin.version>
<guava.version>32.0.1-jre</guava.version>
<guava.version>33.4.8-jre</guava.version>

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.

to be in sync with hadoop-3.5.0

Comment threadpom.xml
<jackson.version>2.18.6</jackson.version>
<jersey.version>1.19.4</jersey.version>
<jettison.version>1.5.4</jettison.version>
<jsr305.version>3.0.0</jsr305.version>

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.

can't find any usage

Comment threadpom.xml
<protobuf.version>3.25.5</protobuf.version>
<protoc-jar-maven-plugin.version>3.11.4</protoc-jar-maven-plugin.version>
<protoc.path>${env.PROTOC_PATH}</protoc.path>
<reload4j.version>1.2.22</reload4j.version>

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.

two version of reload4j was coming 1.2.22 and 1.2.19. Better to define our own version than relying on hadoop.
1.2.19 was coming from slf4j-reload4j dependency

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.

yeah, looks like reload4j 1.2.22 appears in the tez-dist tez.tar.gz after this change, so this looks correct

Comment threadpom.xml
<reload4j.version>1.2.22</reload4j.version>
<restrict-imports.enforcer.version>3.0.0</restrict-imports.enforcer.version>
<roaringbitmap.version>1.2.1</roaringbitmap.version>
<roaringbitmap.version>1.3.0</roaringbitmap.version>

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.

proactively done for iceberg-1.10.x

Comment threadpom.xml Outdated
</dependency>
<dependency>
<groupId>com.fasterxml.jackson</groupId>
<artifactId>jackson-bom</artifactId>

@Aggarwal-RaghavAggarwal-RaghavApr 27, 2026

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.

import.*jackson explicitly used in tez java file but it is relying on hadoop for jackson jars. IMO, this is not correct.

Also 2 version of jackson-core are coming in tez-dist/.../lib 2.14.3 and 2.12.7

 +- org.apache.avro:avro:jar:1.11.4:compile
[INFO] | \- com.fasterxml.jackson.core:jackson-core:jar:2.14.3:compile
INFO] | +- org.apache.hadoop:hadoop-yarn-common:jar:3.4.2:compile
[INFO] | | +- com.sun.jersey:jersey-core:jar:1.19.4:compile
[INFO] | | | \- javax.ws.rs:jsr311-api:jar:1.1.1:compile
[INFO] | | +- com.google.inject.extensions:guice-servlet:jar:4.2.3:compile
[INFO] | | +- com.google.inject:guice:jar:4.2.3:compile
[INFO] | | | +- javax.inject:javax.inject:jar:1:compile
[INFO] | | | \- aopalliance:aopalliance:jar:1.0:compile
[INFO] | | +- com.sun.jersey.contribs:jersey-guice:jar:1.19.4:compile
[INFO] | | +- com.fasterxml.jackson.core:jackson-core:jar:2.12.7:compile

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 0sDocker mode activated.
-1 ❌docker3m 20sDocker failed to build run-specific yetus/tez:tp-7858}.
SubsystemReport/Notes
GITHUB PR#490
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-490/1/console
versionsgit=2.34.1
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@Aggarwal-Raghav

Copy link
Copy Markdown
ContributorAuthor

CC @abstractdog , can you please help with CI privilege issue?

15:01:41 15:01:41 #2 [internal] load metadata for docker.io/yetus/tez:240665345
15:01:41 #2 ERROR: pull access denied, repository does not exist or may require authorization: server message: insufficient_scope: authorization failed

@abstractdog

Copy link
Copy Markdown
Contributor

CC @abstractdog , can you please help with CI privilege issue?

15:01:41 15:01:41 #2 [internal] load metadata for docker.io/yetus/tez:240665345
15:01:41 #2 ERROR: pull access denied, repository does not exist or may require authorization: server message: insufficient_scope: authorization failed

sorry, I forgot about this: so considering the currently ongoing hadoop upgrade and your decent improvements in the precommit area, we can get back to this after those are resolved, if you're okay with it

@Aggarwal-Raghav

Copy link
Copy Markdown
ContributorAuthor

CC @abstractdog , can you please help with CI privilege issue?

15:01:41 15:01:41 #2 [internal] load metadata for docker.io/yetus/tez:240665345
15:01:41 #2 ERROR: pull access denied, repository does not exist or may require authorization: server message: insufficient_scope: authorization failed

sorry, I forgot about this: so considering the currently ongoing hadoop upgrade and your decent improvements in the precommit area, we can get back to this after those are resolved, if you're okay with it

yes, that's the plan. I'll rebase one hadoop3.5.0 is in. The CI failure is because of rebase.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec2m 0sDocker 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 💚mvninstall3m 2smaster passed
+1 💚compile2m 28smaster passed
+1 💚javadoc0m 58smaster passed
_ Patch Compile Tests _
+1 💚mvninstall2m 14sthe patch passed
+1 💚codespell0m 47sNo new issues.
+1 💚compile2m 30sthe patch passed
-1 ❌javac2m 30s/results-compile-javac-root.txtroot generated 12 new + 742 unchanged - 0 fixed = 754 total (was 742)
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚javadoc0m 56sthe patch passed
_ Other Tests _
+1 💚unit57m 12sroot in the patch passed.
+1 💚asflicense0m 35sThe patch does not generate ASF License warnings.
73m 36s
SubsystemReport/Notes
DockerClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-490/2/artifact/out/Dockerfile
Optional Testsdupname compile unit asflicense javac javadoc codespell detsecrets xmllint
unameLinux 5eb7580a8c38 5.15.0-173-generic #183-Ubuntu SMP Fri Mar 6 13:29:34 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitytez-personality.sh
git revisionmaster / 330fdc8
Default JavaEclipse Adoptium-21.0.11+10-LTS
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-490/2/testReport/
Max. process+thread count1448 (vs. ulimit of 5500)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-490/2/console
versionsgit=2.43.0 maven=3.9.15 codespell=2.4.1
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec1m 58sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗detsecrets0m 1sdetect-secrets was not available.
+0 🆗xmllint0m 1sxmllint 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 💚mvninstall3m 4smaster passed
+1 💚compile2m 35smaster passed
+1 💚javadoc0m 58smaster passed
_ Patch Compile Tests _
-1 ❌mvninstall0m 4s/patch-mvninstall-root.txtroot in the patch failed.
+1 💚codespell0m 27sNo new issues.
-1 ❌compile0m 4s/patch-compile-root.txtroot in the patch failed.
-1 ❌javac0m 4s/patch-compile-root.txtroot in the patch failed.
+1 💚blanks0m 0sThe patch has no blanks issues.
-1 ❌javadoc0m 5s/patch-javadoc-root.txtroot in the patch failed.
_ Other Tests _
-1 ❌unit0m 5s/patch-unit-root.txtroot in the patch failed.
+0 🆗asflicense0m 4sASF License check generated no output?
9m 57s
SubsystemReport/Notes
DockerClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-490/3/artifact/out/Dockerfile
Optional Testsdupname compile unit asflicense javac javadoc codespell detsecrets xmllint
unameLinux 00cdf78eccf7 5.15.0-181-generic #191-Ubuntu SMP Fri May 22 19:09:02 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitytez-personality.sh
git revisionmaster / 5766cda
Default JavaEclipse Adoptium-21.0.11+10-LTS
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-490/3/testReport/
Max. process+thread count119 (vs. ulimit of 5500)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-490/3/console
versionsgit=2.43.0 maven=3.9.15 codespell=2.4.1
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 7sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo 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 💚mvninstall3m 4smaster passed
+1 💚compile2m 32smaster passed
+1 💚javadoc0m 58smaster passed
_ Patch Compile Tests _
+1 💚mvninstall2m 11sthe patch passed
+1 💚codespell0m 49sNo new issues.
+1 💚compile2m 32sthe patch passed
-1 ❌javac2m 32s/results-compile-javac-root.txtroot generated 12 new + 707 unchanged - 0 fixed = 719 total (was 707)
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚javadoc0m 51sthe patch passed
_ Other Tests _
+1 💚unit60m 23sroot in the patch passed.
+1 💚asflicense0m 35sThe patch does not generate ASF License warnings.
74m 52s
SubsystemReport/Notes
DockerClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-490/4/artifact/out/Dockerfile
Optional Testsdupname compile unit asflicense javac javadoc codespell detsecrets xmllint
unameLinux 50ed7a0819d2 5.15.0-181-generic #191-Ubuntu SMP Fri May 22 19:09:02 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitytez-personality.sh
git revisionmaster / 5766cda
Default JavaEclipse Adoptium-21.0.11+10-LTS
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-490/4/testReport/
Max. process+thread count1558 (vs. ulimit of 5500)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-490/4/console
versionsgit=2.43.0 maven=3.9.15 codespell=2.4.1
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@Aggarwal-Raghav

Copy link
Copy Markdown
ContributorAuthor

fixed javac in #511

@Aggarwal-Raghav

Copy link
Copy Markdown
ContributorAuthor

@abstractdog , this is ready for review.

@Aggarwal-Raghav

Copy link
Copy Markdown
ContributorAuthor

fixed javac in #511

this is merged. Let me know if I need to rebase and re-trigger the CI ?

@abstractdogabstractdog left a comment

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.

LGTM, feel free to rebase

Comment threadpom.xml
<protobuf.version>3.25.5</protobuf.version>
<protoc-jar-maven-plugin.version>3.11.4</protoc-jar-maven-plugin.version>
<protoc.path>${env.PROTOC_PATH}</protoc.path>
<reload4j.version>1.2.22</reload4j.version>

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.

yeah, looks like reload4j 1.2.22 appears in the tez-dist tez.tar.gz after this change, so this looks correct

@Aggarwal-Raghav

Copy link
Copy Markdown
ContributorAuthor

ok, rebasing this for clean javac yetus

- Update BouncyCastle to 1.82
- Update Guava to 33.4.8-jre
- Update RoaringBitmap to 1.3.0
- Add Jackson 2.18.6 and jackson-bom
- Add Reload4j 1.2.22
- Remove jsr305 dependency version
@Aggarwal-Raghav

Copy link
Copy Markdown
ContributorAuthor

Thanks a lot for review @abstractdog 😀

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec3m 11sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo 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 💚mvninstall3m 15smaster passed
+1 💚compile2m 31smaster passed
+1 💚javadoc1m 0smaster passed
_ Patch Compile Tests _
+1 💚mvninstall2m 11sthe patch passed
+1 💚codespell0m 48sNo new issues.
+1 💚compile2m 29sthe patch passed
+1 💚javac2m 29sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚javadoc0m 55sthe patch passed
_ Other Tests _
+1 💚unit58m 43sroot in the patch passed.
+1 💚asflicense0m 35sThe patch does not generate ASF License warnings.
76m 30s
SubsystemReport/Notes
DockerClientAPI=1.55 ServerAPI=1.55 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-490/5/artifact/out/Dockerfile
Optional Testsdupname compile unit asflicense javac javadoc codespell detsecrets xmllint
unameLinux cf46112098e5 5.15.0-181-generic #191-Ubuntu SMP Fri May 22 19:09:02 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitytez-personality.sh
git revisionmaster / 81e0a24
Default JavaEclipse Adoptium-21.0.11+10-LTS
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-490/5/testReport/
Max. process+thread count1460 (vs. ulimit of 5500)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-490/5/console
versionsgit=2.43.0 maven=3.9.15 codespell=2.4.1
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@abstractdog
abstractdog self-requested a review June 22, 2026 15:49
@abstractdog
abstractdog merged commit f924848 into apache:masterJun 22, 2026
4 checks passed
@Aggarwal-Raghav
Aggarwal-Raghav deleted the TEZ-4713 branch June 28, 2026 17: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