You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tez.apache.org by GitBox <gi...@apache.org> on 2022/02/01 04:53:19 UTC

[GitHub] [tez] tez-yetus commented on pull request #166: TEZ-4227 Introduce convenient methods in TezID subclasses

tez-yetus commented on pull request #166:
URL: https://github.com/apache/tez/pull/166#issuecomment-1026476561


   :confetti_ball: **+1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |  13m  2s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  1s |  No case conflicting files found.  |
   | +1 :green_heart: |  @author  |   0m  0s |  The patch does not contain any @author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  The patch appears to include 20 new or modified test files.  |
   ||| _ master Compile Tests _ |
   | +0 :ok: |  mvndep  |   5m  1s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |   9m 12s |  master passed  |
   | +1 :green_heart: |  compile  |   6m 19s |  master passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04  |
   | +1 :green_heart: |  compile  |   6m  4s |  master passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07  |
   | +1 :green_heart: |  checkstyle  |   6m  1s |  master passed  |
   | +1 :green_heart: |  javadoc  |   5m 48s |  master passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04  |
   | +1 :green_heart: |  javadoc  |   5m 18s |  master passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07  |
   | +0 :ok: |  spotbugs  |   0m 45s |  Used deprecated FindBugs config; considering switching to SpotBugs.  |
   | +1 :green_heart: |  findbugs  |   9m 12s |  master passed  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m  9s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   4m 12s |  the patch passed  |
   | +1 :green_heart: |  compile  |   3m 43s |  the patch passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04  |
   | +1 :green_heart: |  javac  |   3m 43s |  the patch passed  |
   | +1 :green_heart: |  compile  |   3m 25s |  the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07  |
   | +1 :green_heart: |  javac  |   3m 25s |  the patch passed  |
   | -0 :warning: |  checkstyle  |   0m 10s |  tez-common: The patch generated 16 new + 15 unchanged - 0 fixed = 31 total (was 15)  |
   | +1 :green_heart: |  checkstyle  |   0m 12s |  tez-runtime-internals: The patch generated 0 new + 33 unchanged - 1 fixed = 33 total (was 34)  |
   | +1 :green_heart: |  checkstyle  |   0m 13s |  tez-mapreduce: The patch generated 0 new + 55 unchanged - 1 fixed = 55 total (was 56)  |
   | -0 :warning: |  checkstyle  |   1m  7s |  tez-dag: The patch generated 20 new + 1783 unchanged - 20 fixed = 1803 total (was 1803)  |
   | +1 :green_heart: |  checkstyle  |   0m 13s |  tez-tests: The patch generated 0 new + 34 unchanged - 2 fixed = 34 total (was 36)  |
   | +1 :green_heart: |  checkstyle  |   0m 10s |  The patch passed checkstyle in tez-ext-service-tests  |
   | +1 :green_heart: |  checkstyle  |   0m 10s |  The patch passed checkstyle in tez-protobuf-history-plugin  |
   | -0 :warning: |  checkstyle  |   0m 11s |  tez-plugins/tez-yarn-timeline-history: The patch generated 1 new + 58 unchanged - 2 fixed = 59 total (was 60)  |
   | +1 :green_heart: |  checkstyle  |   0m 10s |  The patch passed checkstyle in tez-yarn-timeline-history-with-acls  |
   | +1 :green_heart: |  checkstyle  |   0m  9s |  The patch passed checkstyle in tez-yarn-timeline-cache-plugin  |
   | -0 :warning: |  checkstyle  |   0m 10s |  tez-plugins/tez-yarn-timeline-history-with-fs: The patch generated 1 new + 35 unchanged - 1 fixed = 36 total (was 36)  |
   | +1 :green_heart: |  checkstyle  |   0m 10s |  The patch passed checkstyle in tez-history-parser  |
   | +1 :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace issues.  |
   | +1 :green_heart: |  javadoc  |   2m 59s |  the patch passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04  |
   | +1 :green_heart: |  javadoc  |   2m 42s |  the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07  |
   | +1 :green_heart: |  findbugs  |   7m 55s |  the patch passed  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  |   0m 34s |  tez-common in the patch passed.  |
   | +1 :green_heart: |  unit  |   0m 34s |  tez-runtime-internals in the patch passed.  |
   | +1 :green_heart: |  unit  |   1m 16s |  tez-mapreduce in the patch passed.  |
   | +1 :green_heart: |  unit  |   4m 25s |  tez-dag in the patch passed.  |
   | +1 :green_heart: |  unit  |  38m 50s |  tez-tests in the patch passed.  |
   | +1 :green_heart: |  unit  |   4m  4s |  tez-ext-service-tests in the patch passed.  |
   | +1 :green_heart: |  unit  |   0m 27s |  tez-protobuf-history-plugin in the patch passed.  |
   | +1 :green_heart: |  unit  |   1m 36s |  tez-yarn-timeline-history in the patch passed.  |
   | +1 :green_heart: |  unit  |   1m 50s |  tez-yarn-timeline-history-with-acls in the patch passed.  |
   | +1 :green_heart: |  unit  |   0m 18s |  tez-yarn-timeline-cache-plugin in the patch passed.  |
   | +1 :green_heart: |  unit  |   1m 35s |  tez-yarn-timeline-history-with-fs in the patch passed.  |
   | +1 :green_heart: |  unit  |   2m 29s |  tez-history-parser in the patch passed.  |
   | +1 :green_heart: |  asflicense  |   2m 22s |  The patch does not generate ASF License warnings.  |
   |  |   | 159m  2s |   |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-166/6/artifact/out/Dockerfile |
   | GITHUB PR | https://github.com/apache/tez/pull/166 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile |
   | uname | Linux 11e5938b2063 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | personality/tez.sh |
   | git revision | master / 6d7ef203c |
   | Default Java | Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 |
   | Multi-JDK versions | /usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 |
   | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-166/6/artifact/out/diff-checkstyle-tez-common.txt |
   | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-166/6/artifact/out/diff-checkstyle-tez-dag.txt |
   | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-166/6/artifact/out/diff-checkstyle-tez-plugins_tez-yarn-timeline-history.txt |
   | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-166/6/artifact/out/diff-checkstyle-tez-plugins_tez-yarn-timeline-history-with-fs.txt |
   |  Test Results | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-166/6/testReport/ |
   | Max. process+thread count | 1412 (vs. ulimit of 5500) |
   | modules | C: tez-common tez-runtime-internals tez-mapreduce tez-dag tez-tests tez-ext-service-tests tez-plugins/tez-protobuf-history-plugin tez-plugins/tez-yarn-timeline-history tez-plugins/tez-yarn-timeline-history-with-acls tez-plugins/tez-yarn-timeline-cache-plugin tez-plugins/tez-yarn-timeline-history-with-fs tez-plugins/tez-history-parser U: . |
   | Console output | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-166/6/console |
   | versions | git=2.25.1 maven=3.6.3 findbugs=3.0.1 |
   | Powered by | Apache Yetus 0.12.0 https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@tez.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org