Skip to content

TEZ-4325: Counter for sort time - #143

Open
abstractdog wants to merge 1 commit into
apache:masterfrom
abstractdog:TEZ-4325
Open

TEZ-4325: Counter for sort time#143
abstractdog wants to merge 1 commit into
apache:masterfrom
abstractdog:TEZ-4325

Conversation

@abstractdog

Copy link
Copy Markdown
Contributor

No description provided.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 32sDocker 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 🆗mvndep4m 25sMaven dependency ordering for branch
+1 💚mvninstall9m 3smaster passed
+1 💚compile1m 9smaster passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚compile1m 5smaster passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚checkstyle1m 17smaster passed
+1 💚javadoc1m 18smaster passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚javadoc1m 6smaster passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+0 🆗spotbugs0m 59sUsed deprecated FindBugs config; considering switching to SpotBugs.
+1 💚findbugs2m 23smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 9sMaven dependency ordering for patch
+1 💚mvninstall0m 40sthe patch passed
+1 💚compile0m 43sthe patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚javac0m 43sthe patch passed
+1 💚compile0m 36sthe patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚javac0m 36sthe patch passed
-0 ⚠️checkstyle0m 18stez-runtime-library: The patch generated 1 new + 187 unchanged - 0 fixed = 188 total (was 187)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 40sthe patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚javadoc0m 38sthe patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚findbugs1m 48sthe patch passed
_ Other Tests _
+1 💚unit1m 57stez-api in the patch passed.
+1 💚unit5m 17stez-runtime-library in the patch passed.
+1 💚asflicense0m 22sThe patch does not generate ASF License warnings.
36m 48s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-143/1/artifact/out/Dockerfile
GITHUB PR#143
JIRA IssueTEZ-4325
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile
unameLinux 598bb157e9fa 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / fc63c04
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
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-143/1/artifact/out/diff-checkstyle-tez-runtime-library.txt
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-143/1/testReport/
Max. process+thread count2099 (vs. ulimit of 5500)
modulesC: tez-api tez-runtime-library U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-143/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.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec19m 42sDocker 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 🆗mvndep4m 29sMaven dependency ordering for branch
+1 💚mvninstall9m 24smaster passed
+1 💚compile1m 4smaster passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚compile0m 59smaster passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚checkstyle1m 5smaster passed
+1 💚javadoc1m 10smaster passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚javadoc0m 58smaster passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+0 🆗spotbugs1m 0sUsed deprecated FindBugs config; considering switching to SpotBugs.
+1 💚findbugs2m 22smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 8sMaven dependency ordering for patch
+1 💚mvninstall0m 40sthe patch passed
+1 💚compile0m 45sthe patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚javac0m 45sthe patch passed
+1 💚compile0m 39sthe patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚javac0m 39sthe patch passed
-0 ⚠️checkstyle0m 17stez-runtime-library: The patch generated 1 new + 187 unchanged - 0 fixed = 188 total (was 187)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 40sthe patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚javadoc0m 38sthe patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚findbugs1m 54sthe patch passed
_ Other Tests _
+1 💚unit1m 54stez-api in the patch passed.
+1 💚unit5m 4stez-runtime-library in the patch passed.
+1 💚asflicense0m 19sThe patch does not generate ASF License warnings.
55m 28s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-143/2/artifact/out/Dockerfile
GITHUB PR#143
JIRA IssueTEZ-4325
Optional Testsdupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile
unameLinux 68b847da42c5 4.15.0-142-generic #146-Ubuntu SMP Tue Apr 13 01:11:19 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
checkstylehttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-143/2/artifact/out/diff-checkstyle-tez-runtime-library.txt
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-143/2/testReport/
Max. process+thread count294 (vs. ulimit of 5500)
modulesC: tez-api tez-runtime-library U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-143/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.

@shameersss1

Copy link
Copy Markdown
Contributor

This is a useful counter to debug query perf
+1.

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@shameersss1