TEZ-4703: Append BUILDING.txt to root README.md by Aggarwal-Raghav · Pull Request #477 · apache/tez

Aggarwal-Raghav · 2026-04-10T15:04:20Z

Removed Hadoop2.x sections and updated info about JDK21+ and protobuf version

tez-yetus · 2026-04-10T15:13:59Z

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	8m 4s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
			_ master Compile Tests _
			_ Patch Compile Tests _
+1 💚	codespell	0m 7s		The patch generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	markdownlint	0m 2s		The patch generated 0 new + 0 unchanged - 20 fixed = 0 total (was 20)
			_ Other Tests _
+1 💚	asflicense	0m 26s		The patch does not generate ASF License warnings.
		9m 8s

Subsystem	Report/Notes
Docker	ClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-477/1/artifact/out/Dockerfile
GITHUB PR	#477
Optional Tests	dupname asflicense codespell detsecrets markdownlint
uname	Linux 34525f5bffb4 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	/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-477/src/.yetus/personality.sh
git revision	master / `22d126c`
Max. process+thread count	52 (vs. ulimit of 5500)
modules	C: . U: .
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-477/1/console
versions	git=2.43.0 maven=3.9.14 codespell=2.4.1 markdownlint=0.46.0
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

abstractdog · 2026-04-13T10:12:34Z

-such as Apache Hadoop Map-Reduce, Apache Pig, Apache Hive etc.
+Apache Tez is a generic data-processing pipeline engine envisioned as a
+low-level engine for higher abstractions such as Apache Hadoop Map-Reduce,
+Apache Pig, Apache Hive etc.


nit: let's change the order: Apache Hive, Apache Pig: Hive is the primary upstream I believe, considering the activity of communities
also: I would remove "Apache Hadoop Map-Reduce": Map-Reduce is not a higher abstraction, instead the model

abstractdog · 2026-04-13T10:14:41Z

+To skip tests and Javadocs:
+
+```bash
+mvn package -Dhadoop.version=3.4.2 -DskipTests -Dmaven.javadoc.skip=true


hadoop.version is not relevant here, right?
what about removing "To skip tests and Javadocs:" example? this gives no extra value to the "Building against a Specific Version of Hadoop" section

abstractdog · 2026-04-13T11:37:46Z

+------------
+
+* JDK 21+
+* Maven 3.6.3 or later


bump this according to recent improvements

abstractdog · 2026-04-13T11:38:45Z

+* Maven 3.6.3 or later
+* spotbugs 4.9.3 or later (if running spotbugs)
+* ProtocolBuffer 3.25.5
+* Internet connection for first build (to fetch all dependencies)


we can delete this line now, looks a bit funny: I believe it should be obvious given a maven-based project

abstractdog · 2026-04-13T11:39:57Z

+* Internet connection for first build (to fetch all dependencies)
+* Hadoop 3.x
+
+Maven Modules


I would this whole "Maven Modules" section: this is a list of the modules without any value added

abstractdog · 2026-04-13T11:42:16Z

+Test Options
+------------
+
+* Use `-DskipTests` to skip tests when running Maven goals like `package`,


these are maven related options, nothing to do with Tez, I guess we can remove (as we're not about to provide a generic how-to-maven)

abstractdog · 2026-04-13T11:45:33Z

+For more details, refer to
+[Hadoop Shims](https://cwiki.apache.org/confluence/display/TEZ/HadoopShims).
+
+UI Build Issues


does it make sense to group the UI-related stuff together?
Tez UI
UI Build Issues
Skip UI Build
Issue with PhantomJS on building in PowerPC

abstractdog · 2026-04-13T11:48:02Z

+cd docs
+mvn site


with a single command:
mvn site -pl docs

abstractdog

@Aggarwal-Raghav : thanks for taking care of this so far
I believe, this origin goal is already achieve, I just left some easy-to-handle comments to improve the build information as you're already touching this part, let me know if they make sense

Aggarwal-Raghav · 2026-04-14T08:09:47Z

Thanks for the review @abstractdog , i'll update PR shortly

tez-yetus · 2026-04-14T08:39:57Z

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	7m 9s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
			_ master Compile Tests _
			_ Patch Compile Tests _
+1 💚	codespell	0m 7s		The patch generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	markdownlint	0m 3s		The patch generated 0 new + 0 unchanged - 20 fixed = 0 total (was 20)
			_ Other Tests _
+1 💚	asflicense	0m 25s		The patch does not generate ASF License warnings.
		8m 12s

Subsystem	Report/Notes
Docker	ClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-477/2/artifact/out/Dockerfile
GITHUB PR	#477
Optional Tests	dupname asflicense codespell detsecrets markdownlint
uname	Linux ef4239937c4b 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	/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-477/src/.yetus/personality.sh
git revision	master / `09eefc9`
Max. process+thread count	56 (vs. ulimit of 5500)
modules	C: . U: .
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-477/2/console
versions	git=2.43.0 maven=3.9.14 codespell=2.4.1 markdownlint=0.46.0
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

abstractdog

with the recent changes, this is fine, +1

Aggarwal-Raghav · 2026-04-18T05:39:41Z

Can you please merge this, will update the following PR then

abstractdog reviewed Apr 13, 2026
View reviewed changes

abstractdog self-requested a review April 13, 2026 11:58

abstractdog requested changes Apr 13, 2026
View reviewed changes

Aggarwal-Raghav added 2 commits April 14, 2026 13:41

TEZ-4703: Append BUILDING.txt to root README.md
15c7368

Review comments
09eefc9

Aggarwal-Raghav force-pushed the TEZ-4703 branch from 22d126c to 09eefc9Compare April 14, 2026 08:31

abstractdog mentioned this pull request Apr 14, 2026
TEZ-4706: Use project build directory for Graphviz output and update documentation #480
Merged

abstractdog self-requested a review April 14, 2026 10:18

abstractdog approved these changes Apr 14, 2026
View reviewed changes

abstractdog merged commit 53eb9f5 into apache:masterApr 20, 2026
4 checks passed

Aggarwal-Raghav deleted the TEZ-4703 branch May 1, 2026 05:52

Uh oh!

Conversation

Aggarwal-Raghav commented Apr 10, 2026

Uh oh!

tez-yetus commented Apr 10, 2026

Uh oh!

abstractdogApr 13, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

abstractdogApr 13, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

abstractdogApr 13, 2026

Choose a reason for hiding this comment

Uh oh!

abstractdogApr 13, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

abstractdogApr 13, 2026

Choose a reason for hiding this comment

Uh oh!

abstractdogApr 13, 2026

Choose a reason for hiding this comment

Uh oh!

abstractdogApr 13, 2026

Choose a reason for hiding this comment

Uh oh!

abstractdogApr 13, 2026

Choose a reason for hiding this comment

Uh oh!

abstractdog left a comment

Choose a reason for hiding this comment

Uh oh!

Aggarwal-Raghav commented Apr 14, 2026

Uh oh!

tez-yetus commented Apr 14, 2026

Uh oh!

abstractdog left a comment

Choose a reason for hiding this comment

Uh oh!

Aggarwal-Raghav commented Apr 18, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

abstractdogApr 13, 2026•
edited
Loading

abstractdogApr 13, 2026•
edited
Loading

abstractdogApr 13, 2026•
edited
Loading