Skip to content

TEZ-4732: MergeManager: deadlock when usedMemory exceeds limit but commitMemory is below merge threshold - #515

Merged
abstractdog merged 2 commits into
apache:masterfrom
ryukobayashi:TEZ-4732
Jul 1, 2026
Merged

TEZ-4732: MergeManager: deadlock when usedMemory exceeds limit but commitMemory is below merge threshold#515
abstractdog merged 2 commits into
apache:masterfrom
ryukobayashi:TEZ-4732

Conversation

@ryukobayashi

Copy link
Copy Markdown
Contributor

Details are included on the ticket: https://issues.apache.org/jira/browse/TEZ-4732

@tez-yetus

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 8sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1 💚mvninstall3m 14smaster passed
+1 💚compile2m 33smaster passed
+1 💚checkstyle0m 27smaster passed
+1 💚javadoc0m 25smaster passed
+0 🆗spotbugs0m 50stez-runtime-library in master has 143 extant spotbugs warnings.
_ Patch Compile Tests _
+1 💚mvninstall2m 15sthe patch passed
+1 💚codespell0m 51sNo new issues.
+1 💚compile2m 33sthe patch passed
+1 💚javac2m 33sthe patch passed
+1 💚blanks0m 1sThe patch has no blanks issues.
+1 💚checkstyle0m 22sthe patch passed
+1 💚javadoc0m 24sthe patch passed
+1 💚spotbugs0m 55sthe patch passed
_ Other Tests _
+1 💚unit60m 24sroot in the patch passed.
+1 💚asflicense0m 23sThe patch does not generate ASF License warnings.
76m 48s
SubsystemReport/Notes
DockerClientAPI=1.55 ServerAPI=1.55 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-515/1/artifact/out/Dockerfile
Optional Testsdupname compile unit asflicense javac javadoc spotbugs checkstyle codespell detsecrets
unameLinux 8bf191a847bf 5.15.0-179-generic #189-Ubuntu SMP Tue May 5 18:20:56 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitytez-personality.sh
git revisionmaster / 17d1549
Default JavaEclipse Adoptium-21.0.11+10-LTS
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-515/1/testReport/
Max. process+thread count1377 (vs. ulimit of 5500)
modulesC: tez-runtime-library U: tez-runtime-library
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-515/1/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.

@abstractdog
abstractdog self-requested a review June 30, 2026 14:06

@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.

nice catch @ryukobayashi, thanks for this patch, I only left a minor comment

@tez-yetus

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 8sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 1 new or modified test files.
_ master Compile Tests _
+1 💚mvninstall3m 10smaster passed
+1 💚compile2m 30smaster passed
+1 💚checkstyle0m 26smaster passed
+1 💚javadoc0m 25smaster passed
+0 🆗spotbugs0m 48stez-runtime-library in master has 143 extant spotbugs warnings.
_ Patch Compile Tests _
+1 💚mvninstall2m 11sthe patch passed
+1 💚codespell0m 49sNo new issues.
+1 💚compile2m 31sthe patch passed
+1 💚javac2m 31sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle0m 21sthe patch passed
+1 💚javadoc0m 24sthe patch passed
+1 💚spotbugs0m 52sthe patch passed
_ Other Tests _
+1 💚unit60m 51sroot in the patch passed.
+1 💚asflicense0m 21sThe patch does not generate ASF License warnings.
76m 49s
SubsystemReport/Notes
DockerClientAPI=1.55 ServerAPI=1.55 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-515/2/artifact/out/Dockerfile
Optional Testsdupname compile unit asflicense javac javadoc spotbugs checkstyle codespell detsecrets
unameLinux 90c5002b5c2d 5.15.0-179-generic #189-Ubuntu SMP Tue May 5 18:20:56 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitytez-personality.sh
git revisionmaster / 17d1549
Default JavaEclipse Adoptium-21.0.11+10-LTS
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-515/2/testReport/
Max. process+thread count1563 (vs. ulimit of 5500)
modulesC: tez-runtime-library U: tez-runtime-library
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-515/2/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.

@abstractdog
abstractdog self-requested a review July 1, 2026 08:49
@abstractdog
abstractdog merged commit 8fde353 into apache:masterJul 1, 2026
6 checks passed
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

@ryukobayashi@tez-yetus@abstractdog