Skip to content

TEZ-4502: Migrate to ProtobufRpcEngine2. - #302

Open
ayushtkn wants to merge 1 commit into
apache:masterfrom
ayushtkn:TEZ-4502
Open

TEZ-4502: Migrate to ProtobufRpcEngine2.#302
ayushtkn wants to merge 1 commit into
apache:masterfrom
ayushtkn:TEZ-4502

Conversation

@ayushtkn

Copy link
Copy Markdown
Member

No description provided.

Change-Id: I2517d596725ab304730fdd4682907ba90c8c3260
@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec25m 56sDocker 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 39sMaven dependency ordering for branch
+1 💚mvninstall11m 43smaster passed
+1 💚compile2m 38smaster passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu122.04.1
+1 💚compile2m 29smaster passed with JDK Private Build-1.8.0_362-8u372-gaus1-0ubuntu122.04-b09
+1 💚checkstyle2m 21smaster passed
+1 💚javadoc2m 18smaster passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu122.04.1
+1 💚javadoc1m 56smaster passed with JDK Private Build-1.8.0_362-8u372-gaus1-0ubuntu122.04-b09
+0 🆗spotbugs0m 50sUsed deprecated FindBugs config; considering switching to SpotBugs.
+1 💚findbugs4m 24smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 10sMaven dependency ordering for patch
+1 💚mvninstall1m 23sthe patch passed
+1 💚compile1m 31sthe patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu122.04.1
+1 💚javac1m 31sthe patch passed
+1 💚compile1m 20sthe patch passed with JDK Private Build-1.8.0_362-8u372-gaus1-0ubuntu122.04-b09
+1 💚javac1m 20sthe patch passed
-0 ⚠️checkstyle0m 10stez-common: The patch generated 1 new + 10 unchanged - 0 fixed = 11 total (was 10)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 57sthe patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu122.04.1
+1 💚javadoc0m 55sthe patch passed with JDK Private Build-1.8.0_362-8u372-gaus1-0ubuntu122.04-b09
+1 💚findbugs3m 26sthe patch passed
_ Other Tests _
+1 💚unit2m 19stez-api in the patch passed.
+1 💚unit0m 28stez-common in the patch passed.
-1 ❌unit5m 13stez-dag in the patch failed.
-1 ❌unit1m 9stez-ext-service-tests in the patch failed.
+1 💚asflicense0m 43sThe patch does not generate ASF License warnings.
82m 0s
ReasonTests
Failed junit teststez.dag.app.TestSpeculation
tez.dag.app.TestPreemption
tez.dag.app.TestMockDAGAppMaster
tez.tests.TestExternalTezServicesErrors
tez.tests.TestExtServicesWithLocalMode
tez.tests.TestExternalTezServices
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-302/1/artifact/out/Dockerfile
GITHUB PR#302
JIRA IssueTEZ-4502
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile
unameLinux 6cbe465c42bb 4.15.0-212-generic #223-Ubuntu SMP Tue May 23 13:09:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / a9c5365
Default JavaPrivate Build-1.8.0_362-8u372-gaus1-0ubuntu122.04-b09
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu122.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u372-gaus1-0ubuntu122.04-b09
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-302/1/artifact/out/diff-checkstyle-tez-common.txt
unithttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-302/1/artifact/out/patch-unit-tez-dag.txt
unithttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-302/1/artifact/out/patch-unit-tez-ext-service-tests.txt
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-302/1/testReport/
Max. process+thread count849 (vs. ulimit of 5500)
modulesC: tez-api tez-common tez-dag tez-ext-service-tests U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-302/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.

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

@ayushtkn@tez-yetus