Skip to content

TEZ-4157: ShuffleHandler: upgrade to netty4 - #118

Merged
abstractdog merged 1 commit into
apache:masterfrom
abstractdog:TEZ-4157
Apr 23, 2021
Merged

TEZ-4157: ShuffleHandler: upgrade to netty4#118
abstractdog merged 1 commit into
apache:masterfrom
abstractdog:TEZ-4157

Conversation

@abstractdog

Copy link
Copy Markdown
Contributor

No description provided.

@hadoop-yetus

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec17m 31sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 2 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep5m 57sMaven dependency ordering for branch
+1 💚mvninstall8m 28smaster passed
+1 💚compile3m 3smaster passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚compile2m 45smaster passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+1 💚checkstyle1m 48smaster passed
+1 💚javadoc2m 50smaster passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚javadoc2m 16smaster passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+0 🆗spotbugs5m 56sUsed deprecated FindBugs config; considering switching to SpotBugs.
+0 🆗findbugs5m 51sroot in master has 2 extant findbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for patch
+1 💚mvninstall4m 43sthe patch passed
+1 💚compile3m 2sthe patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚javac3m 2sthe patch passed
+1 💚compile2m 46sthe patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+1 💚javac2m 46sthe patch passed
-0 ⚠️checkstyle0m 18stez-ext-service-tests: The patch generated 4 new + 31 unchanged - 7 fixed = 35 total (was 38)
-0 ⚠️checkstyle0m 18stez-plugins/tez-aux-services: The patch generated 1 new + 66 unchanged - 9 fixed = 67 total (was 75)
-0 ⚠️checkstyle0m 46sroot: The patch generated 5 new + 97 unchanged - 16 fixed = 102 total (was 113)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 4sThe patch has no ill-formed XML file.
+1 💚javadoc2m 40sthe patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚javadoc2m 17sthe patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+1 💚findbugs7m 32sthe patch passed
_ Other Tests _
+1 💚unit4m 0stez-ext-service-tests in the patch passed.
+1 💚unit2m 41stez-aux-services in the patch passed.
+1 💚unit72m 13sroot in the patch passed.
+1 💚asflicense1m 7sThe patch does not generate ASF License warnings.
158m 37s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/1/artifact/out/Dockerfile
GITHUB PR#118
JIRA IssueTEZ-4157
Optional Testsdupname asflicense javac javadoc unit xml compile spotbugs findbugs checkstyle
unameLinux 33d97bd5399b 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / 2b671d6
Default JavaPrivate Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/1/artifact/out/diff-checkstyle-tez-ext-service-tests.txt
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/1/artifact/out/diff-checkstyle-tez-plugins_tez-aux-services.txt
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/1/artifact/out/diff-checkstyle-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/1/testReport/
Max. process+thread count1386 (vs. ulimit of 5500)
modulesC: tez-ext-service-tests tez-plugins/tez-aux-services . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/1/console
versionsgit=2.25.1 maven=3.6.3 findbugs=3.0.1
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@hadoop-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec17m 11sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 2 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep5m 47sMaven dependency ordering for branch
+1 💚mvninstall8m 34smaster passed
+1 💚compile3m 7smaster passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚compile2m 55smaster passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+1 💚checkstyle1m 45smaster passed
+1 💚javadoc3m 2smaster passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚javadoc2m 34smaster passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+0 🆗spotbugs6m 25sUsed deprecated FindBugs config; considering switching to SpotBugs.
+0 🆗findbugs6m 20sroot in master has 2 extant findbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 18sMaven dependency ordering for patch
+1 💚mvninstall5m 47sthe patch passed
+1 💚compile4m 2sthe patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚javac4m 2sthe patch passed
+1 💚compile3m 32sthe patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+1 💚javac3m 32sthe patch passed
-0 ⚠️checkstyle0m 22stez-ext-service-tests: The patch generated 4 new + 31 unchanged - 7 fixed = 35 total (was 38)
-0 ⚠️checkstyle0m 21stez-plugins/tez-aux-services: The patch generated 1 new + 66 unchanged - 9 fixed = 67 total (was 75)
-0 ⚠️checkstyle1m 2sroot: The patch generated 5 new + 97 unchanged - 16 fixed = 102 total (was 113)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 4sThe patch has no ill-formed XML file.
+1 💚javadoc3m 20sthe patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚javadoc2m 49sthe patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+1 💚findbugs9m 59sthe patch passed
_ Other Tests _
-1 ❌unit1m 10stez-ext-service-tests in the patch failed.
+1 💚unit3m 5stez-aux-services in the patch passed.
-1 ❌unit164m 4sroot in the patch failed.
+1 💚asflicense1m 11sThe patch does not generate ASF License warnings.
256m 4s
ReasonTests
Failed junit teststez.tests.TestExternalTezServicesErrors
tez.tests.TestExternalTezServices
tez.test.TestMiniTezCluster
tez.test.TestAMRecovery
tez.test.TestTezJobs
tez.test.TestDAGRecovery
tez.test.TestFaultTolerance
tez.test.TestExceptionPropagation
tez.mapreduce.TestMRRJobsDAGApi
tez.mapreduce.TestMRRJobs
tez.test.TestPipelinedShuffle
tez.test.TestDAGRecovery2
tez.tests.TestExternalTezServicesErrors
tez.tests.TestExternalTezServices
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/2/artifact/out/Dockerfile
GITHUB PR#118
JIRA IssueTEZ-4157
Optional Testsdupname asflicense javac javadoc unit xml compile spotbugs findbugs checkstyle
unameLinux 67f4c0d01b14 4.15.0-126-generic #129-Ubuntu SMP Mon Nov 23 18:53:38 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / aea0707
Default JavaPrivate Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/2/artifact/out/diff-checkstyle-tez-ext-service-tests.txt
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/2/artifact/out/diff-checkstyle-tez-plugins_tez-aux-services.txt
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/2/artifact/out/diff-checkstyle-root.txt
unithttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/2/artifact/out/patch-unit-tez-ext-service-tests.txt
unithttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/2/artifact/out/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/2/testReport/
Max. process+thread count1386 (vs. ulimit of 5500)
modulesC: tez-ext-service-tests tez-plugins/tez-aux-services . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/2/console
versionsgit=2.25.1 maven=3.6.3 findbugs=3.0.1
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@hadoop-yetus

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec24m 2sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 2 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep5m 53sMaven dependency ordering for branch
+1 💚mvninstall8m 47smaster passed
+1 💚compile3m 3smaster passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚compile2m 43smaster passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+1 💚checkstyle1m 47smaster passed
+1 💚javadoc2m 53smaster passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚javadoc2m 18smaster passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+0 🆗spotbugs5m 55sUsed deprecated FindBugs config; considering switching to SpotBugs.
+0 🆗findbugs5m 51sroot in master has 2 extant findbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for patch
+1 💚mvninstall4m 44sthe patch passed
+1 💚compile3m 6sthe patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚javac3m 6sthe patch passed
+1 💚compile2m 52sthe patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+1 💚javac2m 52sthe patch passed
-0 ⚠️checkstyle0m 17stez-ext-service-tests: The patch generated 4 new + 31 unchanged - 7 fixed = 35 total (was 38)
-0 ⚠️checkstyle0m 19stez-plugins/tez-aux-services: The patch generated 1 new + 66 unchanged - 9 fixed = 67 total (was 75)
-0 ⚠️checkstyle0m 47sroot: The patch generated 5 new + 97 unchanged - 16 fixed = 102 total (was 113)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 4sThe patch has no ill-formed XML file.
+1 💚javadoc2m 40sthe patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04
+1 💚javadoc2m 16sthe patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
+1 💚findbugs7m 32sthe patch passed
_ Other Tests _
+1 💚unit3m 56stez-ext-service-tests in the patch passed.
+1 💚unit2m 41stez-aux-services in the patch passed.
+1 💚unit71m 18sroot in the patch passed.
+1 💚asflicense1m 5sThe patch does not generate ASF License warnings.
164m 39s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/3/artifact/out/Dockerfile
GITHUB PR#118
JIRA IssueTEZ-4157
Optional Testsdupname asflicense javac javadoc unit xml compile spotbugs findbugs checkstyle
unameLinux 5dc5fcf555cf 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / aea0707
Default JavaPrivate Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/3/artifact/out/diff-checkstyle-tez-ext-service-tests.txt
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/3/artifact/out/diff-checkstyle-tez-plugins_tez-aux-services.txt
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/3/artifact/out/diff-checkstyle-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/3/testReport/
Max. process+thread count1359 (vs. ulimit of 5500)
modulesC: tez-ext-service-tests tez-plugins/tez-aux-services . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-118/3/console
versionsgit=2.25.1 maven=3.6.3 findbugs=3.0.1
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@abstractdog

Copy link
Copy Markdown
ContributorAuthor

tests passed, I included all the fixes that were done in the scope of HIVE-24524

@abstractdog
abstractdog merged commit 0184781 into apache:masterApr 23, 2021
asfgit pushed a commit that referenced this pull request Apr 23, 2021
…ncy from tez (#118) (Laszlo Bodor reviewed by Ashutosh Chauhan, Jonathan Turner Eagles)
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.

2 participants

@abstractdog@hadoop-yetus