TEZ-4711: Normalize ASF license header by Aggarwal-Raghav · Pull Request #488 · apache/tez

Aggarwal-Raghav · 2026-04-21T18:04:28Z

For yml, sh and dockerfile have manually fixed them. becuase of delimiters and comments in files, there is no generic regex for them.
Maybe this can help: https://github.com/geruh/incubator-xtable/blob/7c7b54b3870791578b4689b29cf8794866eefa52/pom.xml#L542-L633 but it will still require manual intervention at first

tez-yetus · 2026-04-21T18:05:07Z

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 0s		Docker mode activated.
-1 ❌	patch	0m 7s		#488 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem	Report/Notes
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-488/1/console
versions	git=2.34.1
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

tez-yetus · 2026-05-01T16:28:02Z

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 0s		Docker mode activated.
-1 ❌	patch	0m 6s		#488 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem	Report/Notes
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-488/2/console
versions	git=2.34.1
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

abstractdog · 2026-05-02T05:31:44Z

this makes pretty much sense to me, I quite often find myself simply copy the header from an already existing file, so from now on, I'll copy the correct one :)
let me replay the last test job with a workaround to apply large patches (#423 (comment))
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-488/3/

the precommit test is crucial this time even if only comments were changed, because we need to make sure that our checkstyle/spotbugs/etc. think the same about "correct formatting" as us

tez-yetus · 2026-05-02T05:34:23Z

(!) 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-488/3/console in case of problems.

tez-yetus · 2026-05-02T05:37:32Z

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 0s		Docker mode activated.
-1 ❌	docker	3m 7s		Docker failed to build run-specific yetus/tez:tp-20616}.

Subsystem	Report/Notes
GITHUB PR	#488
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-488/3/console
versions	git=2.34.1
Powered by	Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

tez-yetus · 2026-05-02T05:39:05Z

(!) 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-488/4/console in case of problems.

abstractdog · 2026-05-02T05:40:52Z

this makes pretty much sense to me, I quite often find myself simply copy the header from an already existing file, so from now on, I'll copy the correct one :) let me replay the last test job with a workaround to apply large patches (#423 (comment)) https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-488/3/

the precommit test is crucial even if only comments were changed, because we need to make sure that our checkstyle/spotbugs/etc. think the same about "correct formatting" as us

timeout happened in
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-488/3
rerunning as
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-488/4

tez-yetus · 2026-05-02T05:41:19Z

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 0s		Docker mode activated.
-1 ❌	docker	2m 13s		Docker failed to build run-specific yetus/tez:tp-20678}.

Subsystem	Report/Notes
GITHUB PR	#488
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-488/4/console
versions	git=2.34.1
Powered by	Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

tez-yetus · 2026-05-02T05:52:32Z

(!) 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-488/5/console in case of problems.

tez-yetus · 2026-05-02T05:54:46Z

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 0s		Docker mode activated.
-1 ❌	docker	2m 12s		Docker failed to build run-specific yetus/tez:tp-29023}.

Subsystem	Report/Notes
GITHUB PR	#488
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-488/5/console
versions	git=2.34.1
Powered by	Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

tez-yetus · 2026-05-02T08:03:25Z

(!) 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-488/6/console in case of problems.

tez-yetus · 2026-05-02T08:05:38Z

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 0s		Docker mode activated.
-1 ❌	docker	2m 12s		Docker failed to build run-specific yetus/tez:tp-10271}.

Subsystem	Report/Notes
GITHUB PR	#488
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-488/6/console
versions	git=2.34.1
Powered by	Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

tez-yetus · 2026-05-02T18:21:22Z

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 0s		Docker mode activated.
-1 ❌	patch	0m 6s		#488 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem	Report/Notes
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-488/7/console
versions	git=2.34.1
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

Aggarwal-Raghav · 2026-05-02T18:35:22Z

Same error I'm getting locally as well when building docker image.

Maybe with latest change in go repo can you please apply the workaround @abstractdog ? I'm not sure where this go is used anyway? I'm thinking of optimizing this dockerfile

tez-yetus · 2026-05-12T18:36:51Z

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 0s		Docker mode activated.
-1 ❌	patch	0m 8s		#488 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem	Report/Notes
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-488/8/console
versions	git=2.34.1
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

Aggarwal-Raghav · 2026-05-13T06:21:14Z

@abstractdog , whenever you have time can you please trigger that workaround one more time?

tez-yetus · 2026-05-17T05:00:24Z

(!) 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-488/9/console in case of problems.

abstractdog · 2026-05-17T05:00:59Z

(!) 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-488/9/console in case of problems.

replayed as
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-488/9
🤞

Aggarwal-Raghav · 2026-05-17T08:36:25Z

Thanks for triggering it. The reason why the failures are appearing because dev-support is excluded in jenkinsflile because of this flag:
YETUS_ARGS+=("--skip-dirs=dev-support")

[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal com.diffplug.spotless:spotless-maven-plugin:3.1.0:check (spotless-check) on project tez-runtime-internals: Execution spotless-check of goal com.diffplug.spotless:spotless-maven-plugin:3.1.0:check failed: Unable to locate file with path: /home/jenkins/jenkins-home/workspace/tez-multibranch_PR-488/src/tez-runtime-internals/dev-support/spotless/license.java: Could not find resource '/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-488/src/tez-runtime-internals/dev-support/spotless/license.java'. -> [Help 1]
[ERROR]

I have addressed it in #498

Aggarwal-Raghav · 2026-05-17T08:40:15Z

Also all these prev failure that's because of rebase issue is yetus behaviour of checking out PR then master branch and then re-applying the commit. This behaviour is also fixed in #498 by using

                    cd "${WORKSPACE}/${SOURCEDIR}"
                    # Ensure origin/master is available for diffing
                    git fetch origin master || true
                    git diff origin/master...HEAD > "${WORKSPACE}/${PATCHDIR}/local-pr.patch"
                    cd "${WORKSPACE}"

abstractdog · 2026-05-17T16:53:26Z

@Aggarwal-Raghav : shall we go for #498 first and then come back for this one?

Aggarwal-Raghav · 2026-05-17T16:55:05Z

@Aggarwal-Raghav : shall we go for #498 first and then come back for this one?

makes sense

tez-yetus · 2026-05-20T08:31:13Z

(!) 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-488/10/console in case of problems.

Aggarwal-Raghav · 2026-05-20T11:16:27Z

Forgot to fix the -Ptools profile license header 😔

tez-yetus · 2026-05-20T12:15:09Z

(!) 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-488/11/console in case of problems.

Aggarwal-Raghav · 2026-05-20T15:03:47Z

Will take a look at afterwards, the project.root is not resolving correctly. Will eval some plugin to do so.
I'll park it for now and once the other priority PR are merged, will pick it up again.
On bright side, this big PR didn't require the manual workaround in CI to run.

Locally the command is passing 🤔

abstractdog · 2026-05-21T04:40:17Z

Will take a look at afterwards, the project.root is not resolving correctly. Will eval some plugin to do so. I'll park it for now and once the other priority PR are merged, will pick it up again. On bright side, this big PR didn't require the manual workaround in CI to run.

Locally the command is passing 🤔

this big PR didn't require the manual workaround in CI to run. <- exactly, already an important step!

tez-yetus · 2026-05-21T15:50:33Z

(!) 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-488/12/console in case of problems.

Aggarwal-Raghav · 2026-05-21T17:04:49Z

The run was aborted because of infra issue 😓

ERROR: Cannot resume build because FlowNode 35 for FlowHead 1 could not be loaded. This is expected to happen when using the PERFORMANCE_OPTIMIZED durability setting and
     Jenkins is not shut down cleanly. Consider investigating to understand if Jenkins was not shut down cleanly or switching to the MAX_SURVIVABILITY durability setting which
     should prevent this issue in most cases

tez-yetus · 2026-05-25T15:05:30Z

(!) 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-488/13/console in case of problems.

tez-yetus · 2026-05-25T16:55:24Z

(!) 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-488/14/console in case of problems.

tez-yetus · 2026-05-25T16:59:48Z

(!) 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-488/15/console in case of problems.

tez-yetus · 2026-05-25T17:02:44Z

(!) 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-488/16/console in case of problems.

tez-yetus · 2026-05-25T19:54:53Z

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 12s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 8s		No case conflicting files found.
+0 🆗	detsecrets	0m 8s		detect-secrets was not available.
+0 🆗	shelldocs	0m 8s		Shelldocs was not available.
+0 🆗	xmllint	0m 0s		xmllint was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
+1 💚	test4tests	0m 0s		The patch appears to include 280 new or modified test files.
			_ master Compile Tests _
+0 🆗	mvndep	0m 34s		Maven dependency ordering for branch
+1 💚	mvninstall	3m 30s		master passed
+1 💚	compile	2m 42s		master passed
+1 💚	checkstyle	10m 22s		master passed
+1 💚	javadoc	8m 45s		master passed
+0 🆗	spotbugs	0m 31s		hadoop-shim in master has 3 extant spotbugs warnings.
+0 🆗	spotbugs	0m 51s		tez-api in master has 92 extant spotbugs warnings.
+0 🆗	spotbugs	0m 16s		branch/tez-build-tools no spotbugs output file (spotbugsXml.xml)
+0 🆗	spotbugs	0m 33s		tez-common in master has 13 extant spotbugs warnings.
+0 🆗	spotbugs	0m 37s		tez-runtime-internals in master has 92 extant spotbugs warnings.
+0 🆗	spotbugs	0m 44s		tez-runtime-library in master has 143 extant spotbugs warnings.
+0 🆗	spotbugs	0m 37s		tez-mapreduce in master has 97 extant spotbugs warnings.
+0 🆗	spotbugs	0m 30s		tez-examples in master has 2 extant spotbugs warnings.
+0 🆗	spotbugs	1m 5s		tez-dag in master has 537 extant spotbugs warnings.
+0 🆗	spotbugs	0m 34s		tez-tests in master has 6 extant spotbugs warnings.
+0 🆗	spotbugs	0m 32s		branch/tez-ui no spotbugs output file (spotbugsXml.xml)
+0 🆗	spotbugs	1m 6s		tez-plugins in master has 43 extant spotbugs warnings.
+0 🆗	spotbugs	0m 28s		tez-plugins/tez-protobuf-history-plugin in master has 7 extant spotbugs warnings.
+0 🆗	spotbugs	0m 23s		tez-plugins/tez-yarn-timeline-history in master has 2 extant spotbugs warnings.
+0 🆗	spotbugs	0m 22s		tez-plugins/tez-yarn-timeline-history-with-acls in master has 2 extant spotbugs warnings.
+0 🆗	spotbugs	0m 23s		tez-plugins/tez-yarn-timeline-cache-plugin in master has 1 extant spotbugs warnings.
+0 🆗	spotbugs	0m 27s		tez-plugins/tez-yarn-timeline-history-with-fs in master has 5 extant spotbugs warnings.
+0 🆗	spotbugs	0m 28s		tez-plugins/tez-history-parser in master has 21 extant spotbugs warnings.
+0 🆗	spotbugs	0m 30s		tez-plugins/tez-aux-services in master has 5 extant spotbugs warnings.
+0 🆗	spotbugs	0m 40s		tez-tools in master has 26 extant spotbugs warnings.
+0 🆗	spotbugs	0m 25s		tez-tools/analyzers in master has 20 extant spotbugs warnings.
+0 🆗	spotbugs	0m 23s		tez-tools/tez-javadoc-tools in master has 3 extant spotbugs warnings.
+0 🆗	spotbugs	0m 13s		branch/docs no spotbugs output file (spotbugsXml.xml)
+0 🆗	spotbugs	3m 54s		root in master has 1054 extant spotbugs warnings.
+0 🆗	spotbugs	0m 20s		branch/tez-dist no spotbugs output file (spotbugsXml.xml)
+0 🆗	spotbugs	0m 29s		tez-tools/analyzers/job-analyzer in master has 20 extant spotbugs warnings.
+0 🆗	spotbugs	0m 28s		tez-tools/tez-tfile-parser in master has 3 extant spotbugs warnings.
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 9s		Maven dependency ordering for patch
+1 💚	mvninstall	2m 42s		the patch passed
+1 💚	codespell	1m 1s		No new issues.
+1 💚	compile	2m 28s		the patch passed
+1 💚	javac	2m 28s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
-0 ⚠️	checkstyle	0m 12s	/results-checkstyle-hadoop-shim.txt	hadoop-shim: The patch generated 1 new + 11 unchanged - 1 fixed = 12 total (was 12)
-0 ⚠️	checkstyle	0m 20s	/results-checkstyle-tez-common.txt	tez-common: The patch generated 3 new + 110 unchanged - 3 fixed = 113 total (was 113)
-0 ⚠️	checkstyle	0m 22s	/results-checkstyle-tez-runtime-internals.txt	tez-runtime-internals: The patch generated 9 new + 353 unchanged - 9 fixed = 362 total (was 362)
-0 ⚠️	checkstyle	0m 31s	/results-checkstyle-tez-runtime-library.txt	tez-runtime-library: The patch generated 2 new + 1995 unchanged - 2 fixed = 1997 total (was 1997)
-0 ⚠️	checkstyle	0m 39s	/results-checkstyle-tez-dag.txt	tez-dag: The patch generated 15 new + 3000 unchanged - 15 fixed = 3015 total (was 3015)
-0 ⚠️	checkstyle	0m 24s	/results-checkstyle-tez-tests.txt	tez-tests: The patch generated 3 new + 1136 unchanged - 3 fixed = 1139 total (was 1139)
-0 ⚠️	checkstyle	0m 25s	/results-checkstyle-tez-plugins.txt	tez-plugins: The patch generated 10 new + 633 unchanged - 10 fixed = 643 total (was 643)
-0 ⚠️	checkstyle	0m 16s	/results-checkstyle-tez-plugins_tez-protobuf-history-plugin.txt	tez-plugins/tez-protobuf-history-plugin: The patch generated 1 new + 138 unchanged - 1 fixed = 139 total (was 139)
-0 ⚠️	checkstyle	0m 21s	/results-checkstyle-tez-plugins_tez-yarn-timeline-history-with-acls.txt	tez-plugins/tez-yarn-timeline-history-with-acls: The patch generated 1 new + 27 unchanged - 1 fixed = 28 total (was 28)
-0 ⚠️	checkstyle	0m 18s	/results-checkstyle-tez-plugins_tez-yarn-timeline-cache-plugin.txt	tez-plugins/tez-yarn-timeline-cache-plugin: The patch generated 1 new + 20 unchanged - 1 fixed = 21 total (was 21)
-0 ⚠️	checkstyle	0m 18s	/results-checkstyle-tez-plugins_tez-yarn-timeline-history-with-fs.txt	tez-plugins/tez-yarn-timeline-history-with-fs: The patch generated 2 new + 46 unchanged - 2 fixed = 48 total (was 48)
-0 ⚠️	checkstyle	0m 20s	/results-checkstyle-tez-plugins_tez-history-parser.txt	tez-plugins/tez-history-parser: The patch generated 4 new + 222 unchanged - 4 fixed = 226 total (was 226)
-0 ⚠️	checkstyle	0m 17s	/results-checkstyle-tez-plugins_tez-aux-services.txt	tez-plugins/tez-aux-services: The patch generated 1 new + 118 unchanged - 1 fixed = 119 total (was 119)
-0 ⚠️	checkstyle	0m 18s	/results-checkstyle-tez-tools.txt	tez-tools: The patch generated 3 new + 105 unchanged - 3 fixed = 108 total (was 108)
-0 ⚠️	checkstyle	0m 20s	/results-checkstyle-tez-tools_analyzers.txt	tez-tools/analyzers: The patch generated 3 new + 101 unchanged - 3 fixed = 104 total (was 104)
-0 ⚠️	checkstyle	0m 18s	/results-checkstyle-tez-tools_analyzers_job-analyzer.txt	tez-tools/analyzers/job-analyzer: The patch generated 3 new + 101 unchanged - 3 fixed = 104 total (was 104)
-0 ⚠️	checkstyle	0m 15s	/results-checkstyle-hadoop-shim-impls.txt	hadoop-shim-impls: The patch generated 1 new + 1 unchanged - 1 fixed = 2 total (was 2)
-0 ⚠️	checkstyle	0m 18s	/results-checkstyle-hadoop-shim-impls_hadoop-shim-2.8.txt	hadoop-shim-impls/hadoop-shim-2.8: The patch generated 1 new + 1 unchanged - 1 fixed = 2 total (was 2)
-0 ⚠️	checkstyle	1m 23s	/results-checkstyle-root.txt	root: The patch generated 47 new + 10189 unchanged - 47 fixed = 10236 total (was 10236)
+1 💚	hadolint	0m 0s		No new issues.
+1 💚	shellcheck	0m 0s		No new issues.
+1 💚	yamllint	0m 2s		No new issues.
+1 💚	javadoc	9m 41s		the patch passed
+0 🆗	spotbugs	0m 18s		tez-build-tools has no data from spotbugs
+0 🆗	spotbugs	0m 35s		tez-ui has no data from spotbugs
+0 🆗	spotbugs	0m 17s		tez-dist has no data from spotbugs
+0 🆗	spotbugs	0m 19s		docs has no data from spotbugs
			_ Other Tests _
+1 💚	unit	60m 12s		root in the patch passed.
+1 💚	asflicense	7m 26s		The patch does not generate ASF License warnings.
		172m 14s

Subsystem	Report/Notes
Docker	ClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-488/16/artifact/out/Dockerfile
Optional Tests	dupname compile unit asflicense codespell detsecrets hadolint shellcheck shelldocs javac javadoc spotbugs checkstyle xmllint yamllint
uname	Linux abb3b1bb4271 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 tool	maven
Personality	tez-personality.sh
git revision	master / `b6b5d42`
Default Java	Eclipse Adoptium-21.0.11+10-LTS
Test Results	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-488/16/testReport/
Max. process+thread count	1442 (vs. ulimit of 5500)
modules	C: hadoop-shim tez-api tez-build-tools tez-common tez-runtime-internals tez-runtime-library tez-mapreduce tez-examples tez-dag tez-tests tez-ext-service-tests tez-ui tez-plugins tez-plugins/tez-protobuf-history-plugin tez-plugins/tez-yarn-timeline-history tez-plugins/tez-yarn-timeline-history-with-acls tez-plugins/tez-yarn-timeline-cache-plugin tez-plugins/tez-yarn-timeline-history-with-fs tez-plugins/tez-history-parser tez-plugins/tez-aux-services tez-tools tez-tools/analyzers tez-tools/analyzers/job-analyzer tez-tools/tez-javadoc-tools hadoop-shim-impls hadoop-shim-impls/hadoop-shim-2.8 tez-dist docs . tez-tools/tez-tfile-parser U: .
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-488/16/console
versions	git=2.43.0 maven=3.9.15 hadolint=2.14.0 spotbugs=4.9.3 codespell=2.4.1 shellcheck=0.11.0 yamllint=1.38.0
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

Aggarwal-Raghav · 2026-05-25T19:56:28Z

The PR is ready @abstractdog

abstractdog · 2026-05-26T08:18:05Z

is this file needed?

.mvn directory is required for the project.root variable in parent pom.xml to resolve the root direcotry path correctly. In spotbugs and javadoc it is doing cd tez-api; mvn test-compile javadoc:javadoc and the dev-support directory path was not concatenated correctly.
To ensure github clock the empty directory, i kept a dummy empty file .keep in .mvn direcotry.

okay, thanks!

Aggarwal-Raghav mentioned this pull request Apr 21, 2026

HIVE-29455: Normalize Java license headers, part 2 - license source line apache/hive#6313

Open

Aggarwal-Raghav force-pushed the TEZ-4711 branch from 7688137 to 67fbfdc Compare May 1, 2026 16:27

Aggarwal-Raghav force-pushed the TEZ-4711 branch from f368fde to 600dbc5 Compare May 12, 2026 18:36

This comment was marked as outdated.

Sign in to view

Aggarwal-Raghav force-pushed the TEZ-4711 branch from 600dbc5 to 2630eb8 Compare May 20, 2026 08:30

This comment was marked as outdated.

Sign in to view

Aggarwal-Raghav force-pushed the TEZ-4711 branch from 2b0a536 to b966a42 Compare May 25, 2026 15:04

This comment was marked as outdated.

Sign in to view

Aggarwal-Raghav force-pushed the TEZ-4711 branch from 86b19ed to b966a42 Compare May 25, 2026 16:58

TEZ-4711: Normalize ASF license header

d4c1192

Aggarwal-Raghav force-pushed the TEZ-4711 branch from b966a42 to d4c1192 Compare May 25, 2026 17:02

abstractdog reviewed May 26, 2026

View reviewed changes

abstractdog self-requested a review May 26, 2026 09:01

abstractdog approved these changes May 26, 2026

View reviewed changes

abstractdog merged commit 330fdc8 into apache:master May 26, 2026
4 checks passed

Conversation

Aggarwal-Raghav commented Apr 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

tez-yetus commented Apr 21, 2026

Uh oh!

tez-yetus commented May 1, 2026

Uh oh!

abstractdog commented May 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

tez-yetus commented May 2, 2026

Uh oh!

tez-yetus commented May 2, 2026

Uh oh!

tez-yetus commented May 2, 2026

Uh oh!

abstractdog commented May 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

tez-yetus commented May 2, 2026

Uh oh!

tez-yetus commented May 2, 2026

Uh oh!

tez-yetus commented May 2, 2026

Uh oh!

tez-yetus commented May 2, 2026

Uh oh!

tez-yetus commented May 2, 2026

Uh oh!

tez-yetus commented May 2, 2026

Uh oh!

Aggarwal-Raghav commented May 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

tez-yetus commented May 12, 2026

Uh oh!

Aggarwal-Raghav commented May 13, 2026

Uh oh!

tez-yetus commented May 17, 2026

Uh oh!

abstractdog commented May 17, 2026

Uh oh!

This comment was marked as outdated.

Aggarwal-Raghav commented May 17, 2026

Uh oh!

Aggarwal-Raghav commented May 17, 2026

Uh oh!

abstractdog commented May 17, 2026

Uh oh!

Aggarwal-Raghav commented May 17, 2026

Uh oh!

tez-yetus commented May 20, 2026

Uh oh!

This comment was marked as outdated.

Aggarwal-Raghav commented May 20, 2026

Uh oh!

tez-yetus commented May 20, 2026

Uh oh!

This comment was marked as outdated.

Aggarwal-Raghav commented May 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

abstractdog commented May 21, 2026

Uh oh!

tez-yetus commented May 21, 2026

Uh oh!

This comment was marked as outdated.

Aggarwal-Raghav commented May 21, 2026

Uh oh!

tez-yetus commented May 25, 2026

Uh oh!

This comment was marked as outdated.

tez-yetus commented May 25, 2026

Uh oh!

tez-yetus commented May 25, 2026

Uh oh!

tez-yetus commented May 25, 2026

Uh oh!

tez-yetus commented May 25, 2026

Aggarwal-Raghav commented Apr 21, 2026 •

edited

Loading

abstractdog commented May 2, 2026 •

edited

Loading

abstractdog commented May 2, 2026 •

edited

Loading

Aggarwal-Raghav commented May 2, 2026 •

edited

Loading

Aggarwal-Raghav commented May 20, 2026 •

edited

Loading