Skip to content

TEZ-4721: Remove legacy Hadoop 2.x support and hadoop-shim-impls module - #505

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

TEZ-4721: Remove legacy Hadoop 2.x support and hadoop-shim-impls module#505
abstractdog merged 1 commit into
apache:masterfrom
Aggarwal-Raghav:TEZ-4721

Conversation

@Aggarwal-Raghav

@Aggarwal-RaghavAggarwal-Raghav commented Jun 2, 2026

Copy link
Copy Markdown
Contributor
  • Drop Hadoop 2.x Support by eliminate Hadoop Shim Impl module
  • Removed hadoop27, hadoop28 profiles

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec10m 34sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo case conflicting files found.
+0 🆗detsecrets0m 1sdetect-secrets was not available.
+0 🆗xmllint0m 1sxmllint was not available.
+0 🆗shelldocs0m 1sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 32 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep0m 42sMaven dependency ordering for branch
+1 💚mvninstall4m 35smaster passed
+1 💚compile4m 1smaster passed
+1 💚checkstyle10m 42smaster passed
-1 ❌mvnsite0m 43s/branch-mvnsite-tez-api.txttez-api in master failed.
-1 ❌mvnsite0m 40s/branch-mvnsite-tez-runtime-library.txttez-runtime-library in master failed.
-1 ❌mvnsite0m 37s/branch-mvnsite-tez-mapreduce.txttez-mapreduce in master failed.
-1 ❌mvnsite1m 50s/branch-mvnsite-root.txtroot in master failed.
-1 ❌mvnsite0m 27s/branch-mvnsite-hadoop-shim-impls_hadoop-shim-2.7.txthadoop-shim-2.7 in master failed.
-1 ❌javadoc0m 23s/branch-javadoc-hadoop-shim-impls_hadoop-shim-2.7.txthadoop-shim-2.7 in master failed.
+0 🆗spotbugs0m 41shadoop-shim in master has 3 extant spotbugs warnings.
+0 🆗spotbugs1m 18stez-api in master has 92 extant spotbugs warnings.
+0 🆗spotbugs0m 48stez-common in master has 13 extant spotbugs warnings.
+0 🆗spotbugs0m 54stez-runtime-internals in master has 92 extant spotbugs warnings.
+0 🆗spotbugs1m 10stez-runtime-library in master has 143 extant spotbugs warnings.
+0 🆗spotbugs0m 55stez-mapreduce in master has 97 extant spotbugs warnings.
+0 🆗spotbugs0m 45stez-examples in master has 2 extant spotbugs warnings.
+0 🆗spotbugs1m 39stez-dag in master has 537 extant spotbugs warnings.
+0 🆗spotbugs0m 54stez-tests in master has 6 extant spotbugs warnings.
+0 🆗spotbugs1m 51stez-plugins in master has 43 extant spotbugs warnings.
+0 🆗spotbugs0m 47stez-plugins/tez-protobuf-history-plugin in master has 7 extant spotbugs warnings.
+0 🆗spotbugs0m 42stez-plugins/tez-yarn-timeline-history in master has 2 extant spotbugs warnings.
+0 🆗spotbugs0m 41stez-plugins/tez-yarn-timeline-history-with-fs in master has 5 extant spotbugs warnings.
+0 🆗spotbugs0m 50stez-tools/analyzers in master has 20 extant spotbugs warnings.
+0 🆗spotbugs0m 28sbranch/docs no spotbugs output file (spotbugsXml.xml)
+0 🆗spotbugs6m 26sroot in master has 1054 extant spotbugs warnings.
-1 ❌spotbugs0m 24s/branch-spotbugs-hadoop-shim-impls_hadoop-shim-2.7.txthadoop-shim-2.7 in master failed.
+0 🆗spotbugs0m 28sbranch/tez-dist no spotbugs output file (spotbugsXml.xml)
+0 🆗spotbugs0m 45stez-tools/analyzers/job-analyzer in master has 20 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall3m 43sthe patch passed
+1 💚codespell1m 34sThe patch generated 0 new + 29 unchanged - 1 fixed = 29 total (was 30)
+1 💚compile3m 49sthe patch passed
+1 💚javac3m 49sroot generated 0 new + 741 unchanged - 1 fixed = 741 total (was 742)
-1 ❌blanks0m 0s/blanks-eol.txtThe patch has 1 line(s) that end in blanks. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply
+1 💚checkstyle0m 30sThe patch passed checkstyle in tez-api
-0 ⚠️checkstyle0m 26s/results-checkstyle-tez-common.txttez-common: The patch generated 1 new + 2 unchanged - 0 fixed = 3 total (was 2)
+1 💚checkstyle0m 28sThe patch passed checkstyle in tez-runtime-internals
-0 ⚠️checkstyle0m 33s/results-checkstyle-tez-runtime-library.txttez-runtime-library: The patch generated 1 new + 88 unchanged - 0 fixed = 89 total (was 88)
+1 💚checkstyle0m 29sThe patch passed checkstyle in tez-mapreduce
+1 💚checkstyle0m 26sThe patch passed checkstyle in tez-examples
+1 💚checkstyle0m 44stez-dag: The patch generated 0 new + 1138 unchanged - 3 fixed = 1138 total (was 1141)
+1 💚checkstyle0m 28sThe patch passed checkstyle in tez-tests
+1 💚checkstyle0m 26sThe patch passed checkstyle in tez-ext-service-tests
+1 💚checkstyle0m 31sThe patch passed checkstyle in tez-plugins
+1 💚checkstyle0m 26sThe patch passed checkstyle in tez-protobuf-history-plugin
+1 💚checkstyle0m 26sThe patch passed checkstyle in tez-yarn-timeline-history
+1 💚checkstyle0m 27sThe patch passed checkstyle in tez-yarn-timeline-history-with-fs
+1 💚checkstyle0m 26sThe patch passed checkstyle in analyzers
+1 💚checkstyle0m 27sThe patch passed checkstyle in job-analyzer
+1 💚checkstyle0m 23sThe patch passed checkstyle in tez-dist
+1 💚checkstyle0m 21sThe patch passed checkstyle in docs
-0 ⚠️checkstyle1m 9s/results-checkstyle-root.txtroot: The patch generated 2 new + 1459 unchanged - 17 fixed = 1461 total (was 1476)
-1 ❌markdownlint0m 7s/results-markdownlint.txtThe patch generated 1 new + 117 unchanged - 2 fixed = 118 total (was 119)
-1 ❌mvnsite0m 43s/patch-mvnsite-tez-api.txttez-api in the patch failed.
-1 ❌mvnsite0m 39s/patch-mvnsite-tez-runtime-library.txttez-runtime-library in the patch failed.
-1 ❌mvnsite0m 34s/patch-mvnsite-tez-mapreduce.txttez-mapreduce in the patch failed.
-1 ❌mvnsite1m 36s/patch-mvnsite-root.txtroot in the patch failed.
+1 💚shellcheck0m 1sNo new issues.
+1 💚javadoc9m 13sthe patch passed
-1 ❌spotbugs1m 7s/new-spotbugs-tez-runtime-internals.htmltez-runtime-internals generated 7 new + 82 unchanged - 10 fixed = 89 total (was 92)
-1 ❌spotbugs1m 54s/new-spotbugs-tez-dag.htmltez-dag generated 3 new + 533 unchanged - 4 fixed = 536 total (was 537)
+0 🆗spotbugs0m 29stez-dist has no data from spotbugs
+0 🆗spotbugs0m 27sdocs has no data from spotbugs
-1 ❌spotbugs7m 7s/new-spotbugs-root.htmlroot generated 10 new + 1037 unchanged - 17 fixed = 1047 total (was 1054)
_ Other Tests _
+1 💚unit72m 48sroot in the patch passed.
+1 💚asflicense7m 48sThe patch does not generate ASF License warnings.
256m 35s
ReasonTests
SpotBugsmodule:tez-runtime-internals
Exception thrown in class org.apache.tez.runtime.task.TezChild at new org.apache.tez.runtime.task.TezChild(Configuration, String, int, String, String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. At TezChild.java:at new org.apache.tez.runtime.task.TezChild(Configuration, String, int, String, String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. At TezChild.java:[line 189]
new org.apache.tez.runtime.task.TezChild(Configuration, String, int, String, String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) may expose internal representation by storing an externally mutable object into TezChild.credentials At TezChild.java:String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) may expose internal representation by storing an externally mutable object into TezChild.credentials At TezChild.java:[line 149]
new org.apache.tez.runtime.task.TezChild(Configuration, String, int, String, String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) may expose internal representation by storing an externally mutable object into TezChild.defaultConf At TezChild.java:String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) may expose internal representation by storing an externally mutable object into TezChild.defaultConf At TezChild.java:[line 141]
new org.apache.tez.runtime.task.TezChild(Configuration, String, int, String, String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) may expose internal representation by storing an externally mutable object into TezChild.objectRegistry At TezChild.java:String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) may expose internal representation by storing an externally mutable object into TezChild.objectRegistry At TezChild.java:[line 173]
new org.apache.tez.runtime.task.TezChild(Configuration, String, int, String, String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) may expose internal representation by storing an externally mutable object into TezChild.serviceProviderEnvMap At TezChild.java:String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) may expose internal representation by storing an externally mutable object into TezChild.serviceProviderEnvMap At TezChild.java:[line 145]
new org.apache.tez.runtime.task.TezTaskRunner2(Configuration, UserGroupInformation, String[], TaskSpec, int, Map, Map, Multimap, TaskReporterInterface, ExecutorService, ObjectRegistry, String, ExecutionContext, long, boolean, TezExecutors) may expose internal representation by storing an externally mutable object into TezTaskRunner2.taskReporter At TezTaskRunner2.java:int, Map, Map, Multimap, TaskReporterInterface, ExecutorService, ObjectRegistry, String, ExecutionContext, long, boolean, TezExecutors) may expose internal representation by storing an externally mutable object into TezTaskRunner2.taskReporter At TezTaskRunner2.java:[line 142]
new org.apache.tez.runtime.task.TezTaskRunner2(Configuration, UserGroupInformation, String[], TaskSpec, int, Map, Map, Multimap, TaskReporterInterface, ExecutorService, ObjectRegistry, String, ExecutionContext, long, boolean, TezExecutors) may expose internal representation by storing an externally mutable object into TezTaskRunner2.ugi At TezTaskRunner2.java:int, Map, Map, Multimap, TaskReporterInterface, ExecutorService, ObjectRegistry, String, ExecutionContext, long, boolean, TezExecutors) may expose internal representation by storing an externally mutable object into TezTaskRunner2.ugi At TezTaskRunner2.java:[line 141]
SpotBugsmodule:tez-dag
new org.apache.tez.dag.app.rm.TaskSchedulerManager(AppContext, DAGClientServer, EventHandler, ContainerSignatureMatcher, WebUIService, List, boolean) may expose internal representation by storing an externally mutable object into TaskSchedulerManager.appContext At TaskSchedulerManager.java:WebUIService, List, boolean) may expose internal representation by storing an externally mutable object into TaskSchedulerManager.appContext At TaskSchedulerManager.java:[line 191]
new org.apache.tez.dag.app.rm.TaskSchedulerManager(AppContext, DAGClientServer, EventHandler, ContainerSignatureMatcher, WebUIService, List, boolean) may expose internal representation by storing an externally mutable object into TaskSchedulerManager.clientService At TaskSchedulerManager.java:WebUIService, List, boolean) may expose internal representation by storing an externally mutable object into TaskSchedulerManager.clientService At TaskSchedulerManager.java:[line 193]
new org.apache.tez.dag.app.rm.TaskSchedulerManager(AppContext, DAGClientServer, EventHandler, ContainerSignatureMatcher, WebUIService, List, boolean) may expose internal representation by storing an externally mutable object into TaskSchedulerManager.webUI At TaskSchedulerManager.java:WebUIService, List, boolean) may expose internal representation by storing an externally mutable object into TaskSchedulerManager.webUI At TaskSchedulerManager.java:[line 195]
SpotBugsmodule:root
new org.apache.tez.dag.app.rm.TaskSchedulerManager(AppContext, DAGClientServer, EventHandler, ContainerSignatureMatcher, WebUIService, List, boolean) may expose internal representation by storing an externally mutable object into TaskSchedulerManager.appContext At TaskSchedulerManager.java:WebUIService, List, boolean) may expose internal representation by storing an externally mutable object into TaskSchedulerManager.appContext At TaskSchedulerManager.java:[line 191]
new org.apache.tez.dag.app.rm.TaskSchedulerManager(AppContext, DAGClientServer, EventHandler, ContainerSignatureMatcher, WebUIService, List, boolean) may expose internal representation by storing an externally mutable object into TaskSchedulerManager.clientService At TaskSchedulerManager.java:WebUIService, List, boolean) may expose internal representation by storing an externally mutable object into TaskSchedulerManager.clientService At TaskSchedulerManager.java:[line 193]
new org.apache.tez.dag.app.rm.TaskSchedulerManager(AppContext, DAGClientServer, EventHandler, ContainerSignatureMatcher, WebUIService, List, boolean) may expose internal representation by storing an externally mutable object into TaskSchedulerManager.webUI At TaskSchedulerManager.java:WebUIService, List, boolean) may expose internal representation by storing an externally mutable object into TaskSchedulerManager.webUI At TaskSchedulerManager.java:[line 195]
Exception thrown in class org.apache.tez.runtime.task.TezChild at new org.apache.tez.runtime.task.TezChild(Configuration, String, int, String, String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. At TezChild.java:at new org.apache.tez.runtime.task.TezChild(Configuration, String, int, String, String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. At TezChild.java:[line 189]
new org.apache.tez.runtime.task.TezChild(Configuration, String, int, String, String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) may expose internal representation by storing an externally mutable object into TezChild.credentials At TezChild.java:String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) may expose internal representation by storing an externally mutable object into TezChild.credentials At TezChild.java:[line 149]
new org.apache.tez.runtime.task.TezChild(Configuration, String, int, String, String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) may expose internal representation by storing an externally mutable object into TezChild.defaultConf At TezChild.java:String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) may expose internal representation by storing an externally mutable object into TezChild.defaultConf At TezChild.java:[line 141]
new org.apache.tez.runtime.task.TezChild(Configuration, String, int, String, String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) may expose internal representation by storing an externally mutable object into TezChild.objectRegistry At TezChild.java:String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) may expose internal representation by storing an externally mutable object into TezChild.objectRegistry At TezChild.java:[line 173]
new org.apache.tez.runtime.task.TezChild(Configuration, String, int, String, String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) may expose internal representation by storing an externally mutable object into TezChild.serviceProviderEnvMap At TezChild.java:String, int, String, String[], Map, ObjectRegistryImpl, String, ExecutionContext, Credentials, long, String, TezTaskUmbilicalProtocol, boolean) may expose internal representation by storing an externally mutable object into TezChild.serviceProviderEnvMap At TezChild.java:[line 145]
new org.apache.tez.runtime.task.TezTaskRunner2(Configuration, UserGroupInformation, String[], TaskSpec, int, Map, Map, Multimap, TaskReporterInterface, ExecutorService, ObjectRegistry, String, ExecutionContext, long, boolean, TezExecutors) may expose internal representation by storing an externally mutable object into TezTaskRunner2.taskReporter At TezTaskRunner2.java:int, Map, Map, Multimap, TaskReporterInterface, ExecutorService, ObjectRegistry, String, ExecutionContext, long, boolean, TezExecutors) may expose internal representation by storing an externally mutable object into TezTaskRunner2.taskReporter At TezTaskRunner2.java:[line 142]
new org.apache.tez.runtime.task.TezTaskRunner2(Configuration, UserGroupInformation, String[], TaskSpec, int, Map, Map, Multimap, TaskReporterInterface, ExecutorService, ObjectRegistry, String, ExecutionContext, long, boolean, TezExecutors) may expose internal representation by storing an externally mutable object into TezTaskRunner2.ugi At TezTaskRunner2.java:int, Map, Map, Multimap, TaskReporterInterface, ExecutorService, ObjectRegistry, String, ExecutionContext, long, boolean, TezExecutors) may expose internal representation by storing an externally mutable object into TezTaskRunner2.ugi At TezTaskRunner2.java:[line 141]
SubsystemReport/Notes
DockerClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-505/1/artifact/out/Dockerfile
Optional Testsdupname compile unit asflicense codespell detsecrets markdownlint mvnsite xmllint javac javadoc spotbugs checkstyle shellcheck shelldocs
unameLinux 107462f69a36 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-505/1/testReport/
Max. process+thread count2125 (vs. ulimit of 5500)
modulesC: tez-api tez-common tez-runtime-internals tez-runtime-library tez-mapreduce tez-examples tez-dag tez-tests tez-ext-service-tests tez-plugins tez-plugins/tez-protobuf-history-plugin tez-plugins/tez-yarn-timeline-history tez-plugins/tez-yarn-timeline-history-with-fs tez-tools/analyzers tez-tools/analyzers/job-analyzer tez-dist docs . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-505/1/console
versionsgit=2.43.0 maven=3.9.15 spotbugs=4.9.3 codespell=2.4.1 markdownlint=0.46.0 shellcheck=0.11.0
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@abstractdog

Copy link
Copy Markdown
Contributor
  • Drop Hadoop 2.x Support by eliminate Hadoop Shim Layer
  • Moved the Hadoop 2.8+ shims logic to TezUtilsInternal.java.
  • Removed hadoop27, hadoop28 profiles
  • Removed GuavaShim layer as well which was pre v18.0 of guava.

thanks for the patch @Aggarwal-Raghav so far!
dropping the whole shim looks strange for me, I thought this is only for dropping hadoop 2.x, but shims are more generic way to handle different hadoop versions, which can be useful in the future too, right? please let me know if I miss something

the same applies to GuavaShim: it was related to incompatible Guava versions, and I'm afraid removing it can make extra work in the future if we find ourselves in a similar situation...but regardless, removing it should be a scope of a different ticket

this one can focus on dropping hadoop 2.x support only, and we can make it happen quite easily

@Aggarwal-Raghav

Copy link
Copy Markdown
ContributorAuthor
  • Drop Hadoop 2.x Support by eliminate Hadoop Shim Layer
  • Moved the Hadoop 2.8+ shims logic to TezUtilsInternal.java.
  • Removed hadoop27, hadoop28 profiles
  • Removed GuavaShim layer as well which was pre v18.0 of guava.

thanks for the patch @Aggarwal-Raghav so far! dropping the whole shim looks strange for me, I thought this is only for dropping hadoop 2.x, but shims are more generic way to handle different hadoop versions, which can be useful in the future too, right? please let me know if I miss something

the same applies to GuavaShim: it was related to incompatible Guava versions, and I'm afraid removing it can make extra work in the future if we find ourselves in a similar situation...but regardless, removing it should be a scope of a different ticket

this one can focus on dropping hadoop 2.x support only, and we can make it happen quite easily

I thought about it before dropping the shims framework, I considered the potential for future incompatibility scenarios. These include the incompatibility between Hadoop3.x and Hadoop 4.x (in the future maybe). But these scenario are speculative and its better to avoid the maintenance of two versions of the hadoop or guava. It is preferable to support the latest version at the present time.

We had Hadoop27 profile but if you try to run on master it is not even running becuase of self-referene in 1 of the pom.xml. Another instance is that hadoop2.x is jdk8 but now its jdk21. I hightly doubt even with shims present the JVM 8 and JDK 21 JVM cross-compatiblity will work.

But if you think its better to retain shims framework confirm once. I have expressed my thoughts but I don't have the final say here 😅

@Aggarwal-Raghav

Aggarwal-Raghav commented Jun 3, 2026

Copy link
Copy Markdown
ContributorAuthor

ok, I've updated the PR, let me know if its in-line with the scope, I have updated the DefaultHadoopShims to have methods like setHadoopCallerContext , earlier in Hadoop3.x it was no-op. There are few unused fields and methods , which needs discussion.

@abstractdog

Copy link
Copy Markdown
Contributor

ok, I've updated the PR, let me know if its in-line with the scope, I have updated the DefaultHadoopShims to have methods like setHadoopCallerContext , earlier in Hadoop3.x it was no-op. There are few unused fields and methods , which needs discussion.

spot-checked, looks good, let me get back to this in few days in detail
thanks so far!

@Aggarwal-Raghav

Copy link
Copy Markdown
ContributorAuthor

Jenkins stopped the last run, will re-trigger.

01:02:49 ERROR: Caught signal TERM.
01:02:52 01:02:52 01:02:52 Cleaning up docker image used for testing.
01:02:53 Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?
01:02:53 

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 37sDocker 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 appears to include 4 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep0m 40sMaven dependency ordering for branch
+1 💚mvninstall4m 36smaster passed
+1 💚compile4m 2smaster passed
+1 💚checkstyle5m 17smaster passed
-1 ❌mvnsite0m 42s/branch-mvnsite-tez-api.txttez-api in master failed.
-1 ❌mvnsite1m 53s/branch-mvnsite-root.txtroot in master failed.
-1 ❌mvnsite0m 28s/branch-mvnsite-hadoop-shim-impls_hadoop-shim-2.7.txthadoop-shim-2.7 in master failed.
-1 ❌javadoc0m 24s/branch-javadoc-hadoop-shim-impls_hadoop-shim-2.7.txthadoop-shim-2.7 in master failed.
+0 🆗spotbugs0m 42shadoop-shim in master has 3 extant spotbugs warnings.
+0 🆗spotbugs1m 17stez-api in master has 92 extant spotbugs warnings.
+0 🆗spotbugs1m 52stez-plugins in master has 43 extant spotbugs warnings.
+0 🆗spotbugs0m 41stez-plugins/tez-yarn-timeline-history in master has 2 extant spotbugs warnings.
+0 🆗spotbugs0m 49stez-tools/analyzers in master has 20 extant spotbugs warnings.
+0 🆗spotbugs0m 26sbranch/docs no spotbugs output file (spotbugsXml.xml)
+0 🆗spotbugs6m 29sroot in master has 1054 extant spotbugs warnings.
-1 ❌spotbugs0m 25s/branch-spotbugs-hadoop-shim-impls_hadoop-shim-2.7.txthadoop-shim-2.7 in master failed.
+0 🆗spotbugs0m 28sbranch/tez-dist no spotbugs output file (spotbugsXml.xml)
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall3m 43sthe patch passed
+1 💚codespell1m 34sNo new issues.
+1 💚compile3m 56sthe patch passed
+1 💚javac3m 56sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
-0 ⚠️checkstyle0m 25s/results-checkstyle-hadoop-shim.txthadoop-shim: The patch generated 1 new + 8 unchanged - 0 fixed = 9 total (was 8)
-0 ⚠️checkstyle1m 1s/results-checkstyle-root.txtroot: The patch generated 1 new + 8 unchanged - 2 fixed = 9 total (was 10)
+1 💚markdownlint0m 5sThe patch generated 0 new + 117 unchanged - 2 fixed = 117 total (was 119)
-1 ❌mvnsite0m 42s/patch-mvnsite-tez-api.txttez-api in the patch failed.
-1 ❌mvnsite1m 53s/patch-mvnsite-root.txtroot in the patch failed.
+1 💚javadoc4m 49sthe patch passed
+0 🆗spotbugs0m 28stez-dist has no data from spotbugs
+0 🆗spotbugs0m 26sdocs has no data from spotbugs
_ Other Tests _
+1 💚unit71m 58sroot in the patch passed.
+1 💚asflicense4m 9sThe patch does not generate ASF License warnings.
172m 54s
SubsystemReport/Notes
DockerClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-505/3/artifact/out/Dockerfile
Optional Testsdupname compile unit asflicense codespell detsecrets markdownlint mvnsite xmllint javac javadoc spotbugs checkstyle
unameLinux dd47cbdd944c 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-505/3/testReport/
Max. process+thread count1394 (vs. ulimit of 5500)
modulesC: hadoop-shim tez-api tez-plugins tez-plugins/tez-yarn-timeline-history tez-tools/analyzers tez-dist docs . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-505/3/console
versionsgit=2.43.0 maven=3.9.15 spotbugs=4.9.3 codespell=2.4.1 markdownlint=0.46.0
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@Aggarwal-RaghavAggarwal-Raghav changed the title TEZ-4721: Remove legacy Hadoop 2.x support and hadoop-shim modulesTEZ-4721: Remove legacy Hadoop 2.x support and hadoop-shim-impls moduleJun 5, 2026
@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec9m 26sDocker 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 appears to include 4 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep0m 43sMaven dependency ordering for branch
+1 💚mvninstall4m 37smaster passed
+1 💚compile4m 2smaster passed
+1 💚checkstyle5m 7smaster passed
-1 ❌mvnsite0m 26s/branch-mvnsite-hadoop-shim-impls_hadoop-shim-2.7.txthadoop-shim-2.7 in master failed.
-1 ❌javadoc0m 23s/branch-javadoc-hadoop-shim-impls_hadoop-shim-2.7.txthadoop-shim-2.7 in master failed.
+0 🆗spotbugs0m 41shadoop-shim in master has 3 extant spotbugs warnings.
+0 🆗spotbugs1m 17stez-api in master has 92 extant spotbugs warnings.
+0 🆗spotbugs1m 50stez-plugins in master has 43 extant spotbugs warnings.
+0 🆗spotbugs0m 40stez-plugins/tez-yarn-timeline-history in master has 2 extant spotbugs warnings.
+0 🆗spotbugs0m 47stez-tools/analyzers in master has 20 extant spotbugs warnings.
+0 🆗spotbugs0m 25sbranch/docs no spotbugs output file (spotbugsXml.xml)
+0 🆗spotbugs6m 35sroot in master has 1054 extant spotbugs warnings.
-1 ❌spotbugs0m 25s/branch-spotbugs-hadoop-shim-impls_hadoop-shim-2.7.txthadoop-shim-2.7 in master failed.
+0 🆗spotbugs0m 29sbranch/tez-dist no spotbugs output file (spotbugsXml.xml)
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall3m 51sthe patch passed
+1 💚codespell1m 34sNo new issues.
+1 💚compile3m 57sthe patch passed
+1 💚javac3m 57sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
-0 ⚠️checkstyle0m 23s/results-checkstyle-hadoop-shim.txthadoop-shim: The patch generated 1 new + 8 unchanged - 0 fixed = 9 total (was 8)
-0 ⚠️checkstyle0m 59s/results-checkstyle-root.txtroot: The patch generated 1 new + 8 unchanged - 2 fixed = 9 total (was 10)
+1 💚markdownlint0m 5sThe patch generated 0 new + 117 unchanged - 2 fixed = 117 total (was 119)
+1 💚mvnsite21m 47sthe patch passed
+1 💚javadoc4m 42sthe patch passed
+0 🆗spotbugs0m 27stez-dist has no data from spotbugs
+0 🆗spotbugs0m 25sdocs has no data from spotbugs
_ Other Tests _
-1 ❌unit71m 32s/patch-unit-root.txtroot in the patch passed.
+1 💚asflicense3m 57sThe patch does not generate ASF License warnings.
202m 5s
ReasonTests
Failed junit teststez.history.TestHistoryParser
SubsystemReport/Notes
DockerClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-505/4/artifact/out/Dockerfile
Optional Testsdupname compile unit asflicense codespell detsecrets markdownlint mvnsite xmllint javac javadoc spotbugs checkstyle
unameLinux 022764deb853 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-505/4/testReport/
Max. process+thread count2121 (vs. ulimit of 5500)
modulesC: hadoop-shim tez-api tez-plugins tez-plugins/tez-yarn-timeline-history tez-tools/analyzers tez-dist docs . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-505/4/console
versionsgit=2.43.0 maven=3.9.15 spotbugs=4.9.3 codespell=2.4.1 markdownlint=0.46.0
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@Aggarwal-Raghav

Copy link
Copy Markdown
ContributorAuthor
[INFO] -------------------------------------------------------
[INFO] T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.tez.history.TestHistoryParser
[INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 77.63 s -- in org.apache.tez.history.TestHistoryParser
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 01:20 min
[INFO] Finished at: 2026-06-17T14:48:33+05:30
[INFO] ------------------------------------------------------------------------

UT seems flaky, will re-run after review

…ntation
- Retains the core HadoopShim framework and GuavaShim
- Removes hadoop27 and hadoop28 profiles from all pom.xml files
- Removes hadoop-shim-impls module (Hadoop 2.7 and 2.8 specific shims)
@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec9m 19sDocker 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 appears to include 4 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep0m 44sMaven dependency ordering for branch
+1 💚mvninstall4m 32smaster passed
+1 💚compile3m 58smaster passed
+1 💚checkstyle4m 45smaster passed
-1 ❌javadoc0m 23s/branch-javadoc-hadoop-shim-impls_hadoop-shim-2.7.txthadoop-shim-2.7 in master failed.
+0 🆗spotbugs0m 41shadoop-shim in master has 3 extant spotbugs warnings.
+0 🆗spotbugs1m 16stez-api in master has 92 extant spotbugs warnings.
+0 🆗spotbugs1m 49stez-plugins in master has 43 extant spotbugs warnings.
+0 🆗spotbugs0m 41stez-plugins/tez-yarn-timeline-history in master has 2 extant spotbugs warnings.
+0 🆗spotbugs0m 46stez-tools/analyzers in master has 20 extant spotbugs warnings.
+0 🆗spotbugs6m 27sroot in master has 1054 extant spotbugs warnings.
-1 ❌spotbugs0m 24s/branch-spotbugs-hadoop-shim-impls_hadoop-shim-2.7.txthadoop-shim-2.7 in master failed.
+0 🆗spotbugs0m 29sbranch/tez-dist no spotbugs output file (spotbugsXml.xml)
_ Patch Compile Tests _
+0 🆗mvndep0m 14sMaven dependency ordering for patch
+1 💚mvninstall3m 37sthe patch passed
+1 💚codespell1m 33sNo new issues.
+1 💚compile3m 57sthe patch passed
+1 💚javac3m 57sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle0m 22sThe patch passed checkstyle in hadoop-shim
+1 💚checkstyle0m 29sThe patch passed checkstyle in tez-api
+1 💚checkstyle0m 28sThe patch passed checkstyle in tez-plugins
+1 💚checkstyle0m 24sThe patch passed checkstyle in tez-yarn-timeline-history
+1 💚checkstyle0m 25sThe patch passed checkstyle in analyzers
+1 💚checkstyle0m 22sThe patch passed checkstyle in tez-dist
+1 💚checkstyle0m 59sroot: The patch generated 0 new + 8 unchanged - 2 fixed = 8 total (was 10)
+1 💚javadoc4m 16sthe patch passed
+0 🆗spotbugs0m 29stez-dist has no data from spotbugs
_ Other Tests _
+1 💚unit74m 26sroot in the patch passed.
+1 💚asflicense3m 31sThe patch does not generate ASF License warnings.
155m 40s
SubsystemReport/Notes
DockerClientAPI=1.55 ServerAPI=1.55 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-505/5/artifact/out/Dockerfile
Optional Testsdupname compile unit asflicense codespell detsecrets xmllint javac javadoc spotbugs checkstyle
unameLinux 6c35b1835440 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 / 61cc636
Default JavaEclipse Adoptium-21.0.11+10-LTS
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-505/5/testReport/
Max. process+thread count1529 (vs. ulimit of 5500)
modulesC: hadoop-shim tez-api tez-plugins tez-plugins/tez-yarn-timeline-history tez-tools/analyzers tez-dist . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-505/5/console
versionsgit=2.43.0 maven=3.9.15 spotbugs=4.9.3 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

@abstractdog , can you please take a look at this?

@abstractdog
abstractdog self-requested a review June 25, 2026 14:15

@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
tez.tar.gz contents also didn't change only the shim implementation disappeared which was expected

@abstractdog
abstractdog merged commit 9796ac5 into apache:masterJun 25, 2026
6 checks passed
@Aggarwal-Raghav
Aggarwal-Raghav deleted the TEZ-4721 branch June 28, 2026 17:24
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