Skip to content

TEZ-4180: Show convenient input -> output vertex names in output/sort messages - #154

Merged
jteagles merged 1 commit into
apache:masterfrom
abstractdog:TEZ-4180
Oct 14, 2021
Merged

TEZ-4180: Show convenient input -> output vertex names in output/sort messages#154
jteagles merged 1 commit into
apache:masterfrom
abstractdog:TEZ-4180

Conversation

@abstractdog

Copy link
Copy Markdown
Contributor

No description provided.

@tez-yetus

This comment has been minimized.

@tez-yetus

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec17m 27sDocker 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 🆗mvndep4m 25sMaven dependency ordering for branch
+1 💚mvninstall9m 13smaster passed
+1 💚compile1m 57smaster passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚compile1m 47smaster passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚checkstyle1m 42smaster passed
+1 💚javadoc1m 58smaster passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚javadoc1m 42smaster passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+0 🆗spotbugs0m 46sUsed deprecated FindBugs config; considering switching to SpotBugs.
+1 💚findbugs3m 41smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 8sMaven dependency ordering for patch
+1 💚mvninstall1m 13sthe patch passed
+1 💚compile1m 20sthe patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚javac1m 20sthe patch passed
+1 💚compile1m 9sthe patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚javac1m 9sthe patch passed
+1 💚checkstyle0m 12stez-api: The patch generated 0 new + 2 unchanged - 3 fixed = 2 total (was 5)
+1 💚checkstyle0m 9sThe patch passed checkstyle in tez-runtime-internals
+1 💚checkstyle0m 17stez-runtime-library: The patch generated 0 new + 228 unchanged - 9 fixed = 228 total (was 237)
+1 💚checkstyle0m 11sThe patch passed checkstyle in tez-mapreduce
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc1m 11sthe patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚javadoc1m 5sthe patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚findbugs3m 18sthe patch passed
_ Other Tests _
+1 💚unit1m 56stez-api in the patch passed.
+1 💚unit0m 31stez-runtime-internals in the patch passed.
+1 💚unit5m 1stez-runtime-library in the patch passed.
+1 💚unit1m 8stez-mapreduce in the patch passed.
+1 💚asflicense0m 35sThe patch does not generate ASF License warnings.
64m 52s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-154/2/artifact/out/Dockerfile
GITHUB PR#154
JIRA IssueTEZ-4180
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile
unameLinux a8581258fb42 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / c875b82
Default JavaPrivate Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-154/2/testReport/
Max. process+thread count1089 (vs. ulimit of 5500)
modulesC: tez-api tez-runtime-internals tez-runtime-library tez-mapreduce U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-154/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.

@abstractdog

Copy link
Copy Markdown
ContributorAuthor

@jteagles: would you like to review this one? almost trivial logging improvement

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

+1. This is a good change that will help debugging.

@jteagles
jteagles merged commit 58fca8b into apache:masterOct 14, 2021
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

@abstractdog@tez-yetus@jteagles