TEZ-4722: Issue with NODE_TOTAL_COUNT counter

[Aggarwal-Raghav]

Check TEZ-4722

[tez-yetus]

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	8m 43s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	detsecrets	0m 1s		detect-secrets was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
+1 💚	test4tests	0m 0s		The patch appears to include 1 new or modified test files.
			_ master Compile Tests _
+1 💚	mvninstall	5m 10s		master passed
+1 💚	compile	3m 55s		master passed
+1 💚	checkstyle	0m 42s		master passed
+1 💚	javadoc	0m 29s		master passed
+0 🆗	spotbugs	1m 43s		tez-dag in master has 537 extant spotbugs warnings.
			_ Patch Compile Tests _
+1 💚	mvninstall	3m 57s		the patch passed
+1 💚	codespell	1m 46s		No new issues.
+1 💚	compile	3m 55s		the patch passed
+1 💚	javac	3m 55s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	checkstyle	0m 36s		the patch passed
+1 💚	javadoc	0m 27s		the patch passed
+1 💚	spotbugs	1m 54s		the patch passed
			_ Other Tests _
+1 💚	unit	69m 8s		root in the patch passed.
+1 💚	asflicense	0m 31s		The patch does not generate ASF License warnings.
		104m 44s

Subsystem	Report/Notes
Docker	ClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-507/1/artifact/out/Dockerfile
Optional Tests	dupname compile unit asflicense javac javadoc spotbugs checkstyle codespell detsecrets
uname	Linux 447c850009ac 5.15.0-177-generic #187-Ubuntu SMP Sat Apr 11 22:54:33 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	tez-personality.sh
git revision	master / 330fdc8
Default Java	Eclipse Adoptium-21.0.11+10-LTS
Test Results	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-507/1/testReport/
Max. process+thread count	1428 (vs. ulimit of 5500)
modules	C: tez-dag U: tez-dag
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-507/1/console
versions	git=2.43.0 maven=3.9.15 spotbugs=4.9.3 codespell=2.4.1
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

[Aggarwal-Raghav]

CC @abstractdog , can you review this?

[abstractdog]

CC @abstractdog , can you review this?

the change makes sense to me, but I have to double-check it on a cluster where I was able reproduce the problem, let me get back to you in a few days

[Aggarwal-Raghav]

gentle ping for review!

[abstractdog]

tested on cluster, works perfectly after the patch, LGTM