Skip to content

TEZ-4516: Fix log lines with incorrect placeholders. - #310

Merged
abstractdog merged 1 commit into
apache:masterfrom
ayushtkn:TEZ-4516
Oct 5, 2023
Merged

TEZ-4516: Fix log lines with incorrect placeholders.#310
abstractdog merged 1 commit into
apache:masterfrom
ayushtkn:TEZ-4516

Conversation

@ayushtkn

Copy link
Copy Markdown
Member

No description provided.

@tez-yetus

This comment was marked as outdated.

@tez-yetus

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec26m 8sDocker 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 1 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep4m 30sMaven dependency ordering for branch
+1 💚mvninstall6m 42smaster passed
+1 💚compile1m 58smaster passed with JDK Ubuntu-11.0.20.1+1-post-Ubuntu-0ubuntu122.04
+1 💚compile1m 41smaster passed with JDK Private Build-1.8.0_382-8u382-ga-1~22.04.1-b05
+1 💚checkstyle1m 35smaster passed
+1 💚javadoc1m 49smaster passed with JDK Ubuntu-11.0.20.1+1-post-Ubuntu-0ubuntu122.04
+1 💚javadoc1m 19smaster passed with JDK Private Build-1.8.0_382-8u382-ga-1~22.04.1-b05
+0 🆗spotbugs1m 28sUsed deprecated FindBugs config; considering switching to SpotBugs.
+1 💚findbugs3m 40smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall1m 14sthe patch passed
+1 💚compile1m 23sthe patch passed with JDK Ubuntu-11.0.20.1+1-post-Ubuntu-0ubuntu122.04
+1 💚javac1m 23sthe patch passed
+1 💚compile1m 19sthe patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~22.04.1-b05
+1 💚javac1m 19sthe patch passed
+1 💚checkstyle0m 59sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 45sthe patch passed with JDK Ubuntu-11.0.20.1+1-post-Ubuntu-0ubuntu122.04
+1 💚javadoc0m 41sthe patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~22.04.1-b05
+1 💚findbugs3m 34sthe patch passed
_ Other Tests _
+1 💚unit0m 43stez-runtime-internals in the patch passed.
+1 💚unit6m 5stez-runtime-library in the patch passed.
+1 💚unit5m 23stez-dag in the patch passed.
+1 💚asflicense0m 29sThe patch does not generate ASF License warnings.
74m 2s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-310/1/artifact/out/Dockerfile
GITHUB PR#310
JIRA IssueTEZ-4516
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile
unameLinux f0c4032a3eda 4.15.0-213-generic #224-Ubuntu SMP Mon Jun 19 13:30:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / 2ad10b6
Default JavaPrivate Build-1.8.0_382-8u382-ga-1~22.04.1-b05
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.20.1+1-post-Ubuntu-0ubuntu122.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_382-8u382-ga-1~22.04.1-b05
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-310/1/testReport/
Max. process+thread count2100 (vs. ulimit of 5500)
modulesC: tez-runtime-internals tez-runtime-library tez-dag U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-310/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.

@abstractdog
abstractdog self-requested a review October 5, 2023 01:55
@abstractdog
abstractdog merged commit 5bba1ff into apache:masterOct 5, 2023
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

@ayushtkn@tez-yetus@abstractdog