Skip to content

TEZ-4603: Bucket Map Join can hang if the source vertex parallelism i… - #392

Open
mayankkunwar wants to merge 2 commits into
apache:masterfrom
mayankkunwar:TEZ-4603-master
Open

TEZ-4603: Bucket Map Join can hang if the source vertex parallelism i…#392
mayankkunwar wants to merge 2 commits into
apache:masterfrom
mayankkunwar:TEZ-4603-master

Conversation

@mayankkunwar

@mayankkunwarmayankkunwar commented Feb 19, 2025

Copy link
Copy Markdown
Contributor

…s changed by reducer autoparallelism
Description
The changes in this PR are w.r.t. when auto reduce parallelism kicks in, we try to updates downstream vertex's expected number of input, so that the execution hang could be avoided.
In this case, the type of edge between reducer and downstream vertices (Mapper) is a CUSTOM edge.
[Reducer --(C)--> Mapper]
The CUSTOM edge is because of Bucket Map Join

TODO: Unit Tests

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec12m 7sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 1sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+0 🆗mvndep2m 5sMaven dependency ordering for branch
+1 💚mvninstall7m 52smaster passed
+1 💚compile1m 6smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚compile1m 4smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚checkstyle1m 13smaster passed
+1 💚javadoc1m 6smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc1m 3smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+0 🆗spotbugs0m 45sUsed deprecated FindBugs config; considering switching to SpotBugs.
+1 💚findbugs2m 15smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 7sMaven dependency ordering for patch
+1 💚mvninstall0m 45sthe patch passed
+1 💚compile0m 43sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javac0m 43sthe patch passed
+1 💚compile0m 40sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚javac0m 40sthe patch passed
-0 ⚠️checkstyle0m 9stez-runtime-library: The patch generated 1 new + 24 unchanged - 0 fixed = 25 total (was 24)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 33sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc0m 35sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
-1 ❌findbugs0m 45stez-dag generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
_ Other Tests _
+1 💚unit1m 51stez-api in the patch passed.
+1 💚unit4m 19stez-runtime-library in the patch passed.
+1 💚unit4m 10stez-dag in the patch passed.
+1 💚asflicense0m 23sThe patch does not generate ASF License warnings.
47m 29s
ReasonTests
FindBugsmodule:tez-dag
Load of known null value in org.apache.tez.dag.app.dag.impl.VertexImpl.setParallelismWrapper(int, VertexLocationHint, Map, Map, boolean) At VertexImpl.java:in org.apache.tez.dag.app.dag.impl.VertexImpl.setParallelismWrapper(int, VertexLocationHint, Map, Map, boolean) At VertexImpl.java:[line 2004]
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/1/artifact/out/Dockerfile
GITHUB PR#392
JIRA IssueTEZ-4603
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile
unameLinux f3d637dfda40 5.15.0-130-generic #140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / 871866e
Default JavaPrivate Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/1/artifact/out/diff-checkstyle-tez-runtime-library.txt
findbugshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/1/artifact/out/new-findbugs-tez-dag.html
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/1/testReport/
Max. process+thread count2100 (vs. ulimit of 5500)
modulesC: tez-api tez-runtime-library tez-dag U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/1/console
versionsgit=2.34.1 maven=3.6.3 findbugs=3.0.1
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@mayankkunwar
mayankkunwarforce-pushed the TEZ-4603-master branch 4 times, most recently from fe4a672 to 07cfcd5CompareFebruary 19, 2025 10:04
@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 9sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+0 🆗mvndep1m 56sMaven dependency ordering for branch
+1 💚mvninstall7m 43smaster passed
+1 💚compile1m 3smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚compile1m 3smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚checkstyle1m 14smaster passed
+1 💚javadoc1m 0smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc1m 3smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+0 🆗spotbugs0m 42sUsed deprecated FindBugs config; considering switching to SpotBugs.
+1 💚findbugs2m 12smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 7sMaven dependency ordering for patch
+1 💚mvninstall0m 45sthe patch passed
+1 💚compile0m 42sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javac0m 42sthe patch passed
+1 💚compile0m 39sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚javac0m 39sthe patch passed
-0 ⚠️checkstyle0m 9stez-runtime-library: The patch generated 1 new + 24 unchanged - 0 fixed = 25 total (was 24)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 32sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc0m 37sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
-1 ❌findbugs0m 44stez-dag generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
_ Other Tests _
+1 💚unit1m 53stez-api in the patch passed.
+1 💚unit4m 16stez-runtime-library in the patch passed.
+1 💚unit4m 6stez-dag in the patch passed.
+1 💚asflicense0m 23sThe patch does not generate ASF License warnings.
34m 55s
ReasonTests
FindBugsmodule:tez-dag
Load of known null value in org.apache.tez.dag.app.dag.impl.VertexImpl.setParallelismWrapper(int, VertexLocationHint, Map, Map, boolean) At VertexImpl.java:in org.apache.tez.dag.app.dag.impl.VertexImpl.setParallelismWrapper(int, VertexLocationHint, Map, Map, boolean) At VertexImpl.java:[line 2004]
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/2/artifact/out/Dockerfile
GITHUB PR#392
JIRA IssueTEZ-4603
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile
unameLinux 527e63d5e93b 5.15.0-130-generic #140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / 871866e
Default JavaPrivate Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/2/artifact/out/diff-checkstyle-tez-runtime-library.txt
findbugshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/2/artifact/out/new-findbugs-tez-dag.html
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/2/testReport/
Max. process+thread count2100 (vs. ulimit of 5500)
modulesC: tez-api tez-runtime-library tez-dag U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/2/console
versionsgit=2.34.1 maven=3.6.3 findbugs=3.0.1
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 10sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+0 🆗mvndep1m 54sMaven dependency ordering for branch
+1 💚mvninstall7m 51smaster passed
+1 💚compile1m 5smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚compile1m 0smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚checkstyle1m 2smaster passed
+1 💚javadoc0m 54smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc0m 46smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+0 🆗spotbugs0m 41sUsed deprecated FindBugs config; considering switching to SpotBugs.
+1 💚findbugs2m 6smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 6sMaven dependency ordering for patch
+1 💚mvninstall0m 44sthe patch passed
+1 💚compile0m 50sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javac0m 50sthe patch passed
+1 💚compile0m 41sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚javac0m 41sthe patch passed
-0 ⚠️checkstyle0m 8stez-runtime-library: The patch generated 1 new + 24 unchanged - 0 fixed = 25 total (was 24)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 31sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc0m 32sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
-1 ❌findbugs0m 46stez-dag generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
_ Other Tests _
+1 💚unit1m 50stez-api in the patch passed.
+1 💚unit4m 19stez-runtime-library in the patch passed.
+1 💚unit4m 5stez-dag in the patch passed.
+1 💚asflicense0m 24sThe patch does not generate ASF License warnings.
34m 17s
ReasonTests
FindBugsmodule:tez-dag
Load of known null value in org.apache.tez.dag.app.dag.impl.VertexImpl.setParallelismWrapper(int, VertexLocationHint, Map, Map, boolean) At VertexImpl.java:in org.apache.tez.dag.app.dag.impl.VertexImpl.setParallelismWrapper(int, VertexLocationHint, Map, Map, boolean) At VertexImpl.java:[line 2004]
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/3/artifact/out/Dockerfile
GITHUB PR#392
JIRA IssueTEZ-4603
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile
unameLinux 52670823285b 5.15.0-130-generic #140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / 871866e
Default JavaPrivate Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/3/artifact/out/diff-checkstyle-tez-runtime-library.txt
findbugshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/3/artifact/out/new-findbugs-tez-dag.html
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/3/testReport/
Max. process+thread count2101 (vs. ulimit of 5500)
modulesC: tez-api tez-runtime-library tez-dag U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/3/console
versionsgit=2.34.1 maven=3.6.3 findbugs=3.0.1
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 9sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+0 🆗mvndep1m 53sMaven dependency ordering for branch
+1 💚mvninstall7m 52smaster passed
+1 💚compile0m 58smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚compile0m 53smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚checkstyle1m 2smaster passed
+1 💚javadoc0m 56smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc0m 48smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+0 🆗spotbugs0m 46sUsed deprecated FindBugs config; considering switching to SpotBugs.
+1 💚findbugs2m 19smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 6sMaven dependency ordering for patch
+1 💚mvninstall0m 43sthe patch passed
+1 💚compile0m 49sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javac0m 49sthe patch passed
+1 💚compile0m 44sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚javac0m 44sthe patch passed
-0 ⚠️checkstyle0m 8stez-runtime-library: The patch generated 1 new + 24 unchanged - 0 fixed = 25 total (was 24)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 34sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc0m 33sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
-1 ❌findbugs0m 48stez-dag generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
_ Other Tests _
+1 💚unit1m 48stez-api in the patch passed.
+1 💚unit4m 22stez-runtime-library in the patch passed.
+1 💚unit4m 7stez-dag in the patch passed.
+1 💚asflicense0m 23sThe patch does not generate ASF License warnings.
34m 38s
ReasonTests
FindBugsmodule:tez-dag
Load of known null value in org.apache.tez.dag.app.dag.impl.VertexImpl.setParallelismWrapper(int, VertexLocationHint, Map, Map, boolean) At VertexImpl.java:in org.apache.tez.dag.app.dag.impl.VertexImpl.setParallelismWrapper(int, VertexLocationHint, Map, Map, boolean) At VertexImpl.java:[line 2004]
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/4/artifact/out/Dockerfile
GITHUB PR#392
JIRA IssueTEZ-4603
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile
unameLinux 8426e7be2a34 5.15.0-130-generic #140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / 871866e
Default JavaPrivate Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/4/artifact/out/diff-checkstyle-tez-runtime-library.txt
findbugshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/4/artifact/out/new-findbugs-tez-dag.html
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/4/testReport/
Max. process+thread count2100 (vs. ulimit of 5500)
modulesC: tez-api tez-runtime-library tez-dag U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/4/console
versionsgit=2.34.1 maven=3.6.3 findbugs=3.0.1
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 10sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+0 🆗mvndep1m 52sMaven dependency ordering for branch
+1 💚mvninstall7m 46smaster passed
+1 💚compile1m 3smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚compile1m 2smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚checkstyle1m 1smaster passed
+1 💚javadoc0m 56smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc0m 52smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+0 🆗spotbugs0m 44sUsed deprecated FindBugs config; considering switching to SpotBugs.
+1 💚findbugs2m 16smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 8sMaven dependency ordering for patch
+1 💚mvninstall0m 49sthe patch passed
+1 💚compile0m 49sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javac0m 49sthe patch passed
+1 💚compile0m 42sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚javac0m 42sthe patch passed
-0 ⚠️checkstyle0m 10stez-runtime-library: The patch generated 1 new + 24 unchanged - 0 fixed = 25 total (was 24)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 31sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc0m 35sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚findbugs2m 8sthe patch passed
_ Other Tests _
+1 💚unit1m 47stez-api in the patch passed.
+1 💚unit4m 26stez-runtime-library in the patch passed.
+1 💚unit4m 13stez-dag in the patch passed.
+1 💚asflicense0m 24sThe patch does not generate ASF License warnings.
35m 9s
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/5/artifact/out/Dockerfile
GITHUB PR#392
JIRA IssueTEZ-4603
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile
unameLinux 82fb6da6ccd2 5.15.0-130-generic #140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / 871866e
Default JavaPrivate Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/5/artifact/out/diff-checkstyle-tez-runtime-library.txt
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/5/testReport/
Max. process+thread count1100 (vs. ulimit of 5500)
modulesC: tez-api tez-runtime-library tez-dag U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/5/console
versionsgit=2.34.1 maven=3.6.3 findbugs=3.0.1
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 9sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+0 🆗mvndep1m 54sMaven dependency ordering for branch
+1 💚mvninstall7m 40smaster passed
+1 💚compile1m 6smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚compile1m 5smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚checkstyle1m 10smaster passed
+1 💚javadoc1m 10smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc1m 2smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+0 🆗spotbugs0m 47sUsed deprecated FindBugs config; considering switching to SpotBugs.
+1 💚findbugs2m 16smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 7sMaven dependency ordering for patch
+1 💚mvninstall0m 45sthe patch passed
+1 💚compile0m 43sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javac0m 43sthe patch passed
+1 💚compile0m 43sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚javac0m 43sthe patch passed
-0 ⚠️checkstyle0m 8stez-runtime-library: The patch generated 1 new + 24 unchanged - 0 fixed = 25 total (was 24)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 32sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc0m 36sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚findbugs1m 50sthe patch passed
_ Other Tests _
+1 💚unit1m 53stez-api in the patch passed.
+1 💚unit4m 17stez-runtime-library in the patch passed.
+1 💚unit4m 11stez-dag in the patch passed.
+1 💚asflicense0m 23sThe patch does not generate ASF License warnings.
35m 7s
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/6/artifact/out/Dockerfile
GITHUB PR#392
JIRA IssueTEZ-4603
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile
unameLinux b6a261420f20 5.15.0-130-generic #140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / 871866e
Default JavaPrivate Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/6/artifact/out/diff-checkstyle-tez-runtime-library.txt
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/6/testReport/
Max. process+thread count1100 (vs. ulimit of 5500)
modulesC: tez-api tez-runtime-library tez-dag U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/6/console
versionsgit=2.34.1 maven=3.6.3 findbugs=3.0.1
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 10sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+0 🆗mvndep1m 53sMaven dependency ordering for branch
+1 💚mvninstall7m 42smaster passed
+1 💚compile1m 2smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚compile1m 0smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚checkstyle1m 10smaster passed
+1 💚javadoc1m 6smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc1m 1smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+0 🆗spotbugs0m 46sUsed deprecated FindBugs config; considering switching to SpotBugs.
+1 💚findbugs2m 18smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 7sMaven dependency ordering for patch
+1 💚mvninstall0m 45sthe patch passed
+1 💚compile0m 43sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javac0m 43sthe patch passed
+1 💚compile0m 40sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚javac0m 40sthe patch passed
-0 ⚠️checkstyle0m 9stez-runtime-library: The patch generated 1 new + 24 unchanged - 0 fixed = 25 total (was 24)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 31sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc0m 34sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚findbugs1m 57sthe patch passed
_ Other Tests _
+1 💚unit1m 52stez-api in the patch passed.
+1 💚unit4m 21stez-runtime-library in the patch passed.
+1 💚unit4m 11stez-dag in the patch passed.
+1 💚asflicense0m 23sThe patch does not generate ASF License warnings.
35m 8s
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/7/artifact/out/Dockerfile
GITHUB PR#392
JIRA IssueTEZ-4603
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile
unameLinux 22d76683320a 5.15.0-130-generic #140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / 871866e
Default JavaPrivate Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/7/artifact/out/diff-checkstyle-tez-runtime-library.txt
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/7/testReport/
Max. process+thread count2100 (vs. ulimit of 5500)
modulesC: tez-api tez-runtime-library tez-dag U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/7/console
versionsgit=2.34.1 maven=3.6.3 findbugs=3.0.1
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec14m 5sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+0 🆗mvndep2m 0sMaven dependency ordering for branch
+1 💚mvninstall7m 44smaster passed
+1 💚compile1m 5smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚compile1m 5smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚checkstyle1m 11smaster passed
+1 💚javadoc1m 3smaster passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc1m 0smaster passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+0 🆗spotbugs0m 45sUsed deprecated FindBugs config; considering switching to SpotBugs.
+1 💚findbugs2m 13smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 8sMaven dependency ordering for patch
+1 💚mvninstall0m 42sthe patch passed
+1 💚compile0m 43sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javac0m 43sthe patch passed
+1 💚compile0m 42sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚javac0m 42sthe patch passed
-0 ⚠️checkstyle0m 9stez-runtime-library: The patch generated 1 new + 24 unchanged - 0 fixed = 25 total (was 24)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 31sthe patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚javadoc0m 36sthe patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚findbugs1m 53sthe patch passed
_ Other Tests _
+1 💚unit1m 52stez-api in the patch passed.
+1 💚unit4m 19stez-runtime-library in the patch passed.
+1 💚unit4m 10stez-dag in the patch passed.
+1 💚asflicense0m 24sThe patch does not generate ASF License warnings.
49m 2s
SubsystemReport/Notes
DockerClientAPI=1.48 ServerAPI=1.48 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/8/artifact/out/Dockerfile
GITHUB PR#392
JIRA IssueTEZ-4603
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile
unameLinux b4171bf66d1f 5.15.0-130-generic #140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / 3aa2aa1
Default JavaPrivate Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/8/artifact/out/diff-checkstyle-tez-runtime-library.txt
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/8/testReport/
Max. process+thread count1100 (vs. ulimit of 5500)
modulesC: tez-api tez-runtime-library tez-dag U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-392/8/console
versionsgit=2.34.1 maven=3.6.3 findbugs=3.0.1
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@okumin

Copy link
Copy Markdown
Contributor

@mayankkunwar
I recently resolved another hang issue with additional integration tests. This change can be a reference for your work.
44c4f1e#diff-ad65a331fa51a07f3cc5301ca7df09c199e9730a6f889bc8b1859554ccfc0519

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

@mayankkunwar@tez-yetus@okumin