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/03/31 16:29:16 UTC
[GitHub] [tez] tez-yetus commented on pull request #197: TEZ-4396 Ensure utility classes have only private (default) constructors
tez-yetus commented on pull request #197:
URL: https://github.com/apache/tez/pull/197#issuecomment-1084818557
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 12m 53s | 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 0s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 8m 58s | master passed |
| +1 :green_heart: | compile | 6m 59s | master passed with JDK Ubuntu-11.0.14.1+1-Ubuntu-0ubuntu1.20.04 |
| +1 :green_heart: | compile | 6m 41s | 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 | 6m 31s | master passed with JDK Ubuntu-11.0.14.1+1-Ubuntu-0ubuntu1.20.04 |
| +1 :green_heart: | javadoc | 6m 0s | 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. |
| +0 :ok: | findbugs | 0m 42s | tez-tools/analyzers/job-analyzer in master has 4 extant findbugs warnings. |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 9s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 4m 24s | the patch passed |
| +1 :green_heart: | compile | 4m 10s | the patch passed with JDK Ubuntu-11.0.14.1+1-Ubuntu-0ubuntu1.20.04 |
| +1 :green_heart: | javac | 4m 10s | the patch passed |
| +1 :green_heart: | compile | 3m 51s | the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 |
| +1 :green_heart: | javac | 3m 51s | the patch passed |
| -0 :warning: | checkstyle | 0m 17s | tez-api: The patch generated 12 new + 161 unchanged - 16 fixed = 173 total (was 177) |
| -0 :warning: | checkstyle | 0m 11s | tez-common: The patch generated 3 new + 53 unchanged - 5 fixed = 56 total (was 58) |
| -0 :warning: | checkstyle | 0m 11s | tez-runtime-internals: The patch generated 4 new + 20 unchanged - 4 fixed = 24 total (was 24) |
| -0 :warning: | checkstyle | 0m 23s | tez-runtime-library: The patch generated 42 new + 244 unchanged - 58 fixed = 286 total (was 302) |
| -0 :warning: | checkstyle | 0m 15s | tez-mapreduce: The patch generated 15 new + 93 unchanged - 19 fixed = 108 total (was 112) |
| -0 :warning: | checkstyle | 0m 9s | tez-examples: The patch generated 1 new + 2 unchanged - 13 fixed = 3 total (was 15) |
| -0 :warning: | checkstyle | 0m 23s | tez-dag: The patch generated 13 new + 152 unchanged - 16 fixed = 165 total (was 168) |
| -0 :warning: | checkstyle | 0m 16s | tez-tests: The patch generated 8 new + 101 unchanged - 13 fixed = 109 total (was 114) |
| -0 :warning: | checkstyle | 0m 10s | tez-ext-service-tests: The patch generated 2 new + 7 unchanged - 2 fixed = 9 total (was 9) |
| -0 :warning: | checkstyle | 0m 11s | tez-plugins/tez-yarn-timeline-history: The patch generated 1 new + 44 unchanged - 1 fixed = 45 total (was 45) |
| -0 :warning: | checkstyle | 0m 10s | tez-plugins/tez-history-parser: The patch generated 1 new + 3 unchanged - 1 fixed = 4 total (was 4) |
| -0 :warning: | checkstyle | 0m 11s | tez-tools/analyzers/job-analyzer: The patch generated 2 new + 2 unchanged - 2 fixed = 4 total (was 4) |
| -0 :warning: | checkstyle | 0m 9s | tez-tools/tez-javadoc-tools: The patch generated 1 new + 2 unchanged - 2 fixed = 3 total (was 4) |
| +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. |
| +1 :green_heart: | javadoc | 3m 30s | the patch passed with JDK Ubuntu-11.0.14.1+1-Ubuntu-0ubuntu1.20.04 |
| +1 :green_heart: | javadoc | 3m 12s | the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 |
| +1 :green_heart: | findbugs | 9m 15s | the patch passed |
||| _ Other Tests _ |
| +1 :green_heart: | unit | 2m 5s | tez-api in the patch passed. |
| +1 :green_heart: | unit | 0m 27s | tez-common in the patch passed. |
| +1 :green_heart: | unit | 0m 34s | tez-runtime-internals in the patch passed. |
| +1 :green_heart: | unit | 4m 36s | tez-runtime-library in the patch passed. |
| +1 :green_heart: | unit | 1m 15s | tez-mapreduce in the patch passed. |
| +1 :green_heart: | unit | 0m 16s | tez-examples in the patch passed. |
| +1 :green_heart: | unit | 4m 27s | tez-dag in the patch passed. |
| -1 :x: | unit | 35m 57s | tez-tests in the patch failed. |
| +1 :green_heart: | unit | 4m 5s | tez-ext-service-tests in the patch passed. |
| +1 :green_heart: | unit | 1m 36s | tez-yarn-timeline-history in the patch passed. |
| +1 :green_heart: | unit | 2m 26s | tez-history-parser in the patch passed. |
| +1 :green_heart: | unit | 2m 45s | job-analyzer in the patch passed. |
| +1 :green_heart: | unit | 0m 16s | tez-javadoc-tools in the patch passed. |
| +1 :green_heart: | asflicense | 2m 34s | The patch does not generate ASF License warnings. |
| | | 169m 1s | |
| Reason | Tests |
|-------:|:------|
| Failed junit tests | tez.test.TestRecovery |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-197/1/artifact/out/Dockerfile |
| GITHUB PR | https://github.com/apache/tez/pull/197 |
| Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile |
| uname | Linux 6a39d4f54a28 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 / f724c5460 |
| 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.14.1+1-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-197/1/artifact/out/diff-checkstyle-tez-api.txt |
| checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-197/1/artifact/out/diff-checkstyle-tez-common.txt |
| checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-197/1/artifact/out/diff-checkstyle-tez-runtime-internals.txt |
| checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-197/1/artifact/out/diff-checkstyle-tez-runtime-library.txt |
| checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-197/1/artifact/out/diff-checkstyle-tez-mapreduce.txt |
| checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-197/1/artifact/out/diff-checkstyle-tez-examples.txt |
| checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-197/1/artifact/out/diff-checkstyle-tez-dag.txt |
| checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-197/1/artifact/out/diff-checkstyle-tez-tests.txt |
| checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-197/1/artifact/out/diff-checkstyle-tez-ext-service-tests.txt |
| checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-197/1/artifact/out/diff-checkstyle-tez-plugins_tez-yarn-timeline-history.txt |
| checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-197/1/artifact/out/diff-checkstyle-tez-plugins_tez-history-parser.txt |
| checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-197/1/artifact/out/diff-checkstyle-tez-tools_analyzers_job-analyzer.txt |
| checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-197/1/artifact/out/diff-checkstyle-tez-tools_tez-javadoc-tools.txt |
| unit | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-197/1/artifact/out/patch-unit-tez-tests.txt |
| Test Results | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-197/1/testReport/ |
| Max. process+thread count | 1326 (vs. ulimit of 5500) |
| modules | C: tez-api tez-common tez-runtime-internals tez-runtime-library tez-mapreduce tez-examples tez-dag tez-tests tez-ext-service-tests tez-plugins/tez-yarn-timeline-history tez-plugins/tez-history-parser tez-tools/analyzers/job-analyzer tez-tools/tez-javadoc-tools U: . |
| Console output | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-197/1/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