tez-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tez CI (Jira)" <j...@apache.org>
Subject [jira] [Commented] (TEZ-4095) Review of Debug Logging
Date Tue, 19 Jan 2021 21:59:00 GMT

    [ https://issues.apache.org/jira/browse/TEZ-4095?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17268256#comment-17268256
] 

Tez CI commented on TEZ-4095:
-----------------------------

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 11m 12s{color} | {color:blue}
Docker mode activated. {color} |
|| || || || {color:brown} Prechecks {color} ||
| {color:green}+1{color} | {color:green} dupname {color} | {color:green}  0m  1s{color} |
{color:green} No case conflicting files found. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  0s{color} |
{color:green} The patch does not contain any @author tags. {color} |
| {color:red}-1{color} | {color:red} test4tests {color} | {color:red}  0m  0s{color} | {color:red}
The 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. {color} |
|| || || || {color:brown} master Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  4m 48s{color} | {color:blue}
Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  7m 13s{color}
| {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  3m 39s{color} |
{color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  4m 12s{color}
| {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  3m 28s{color} |
{color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} spotbugs {color} | {color:blue}  0m 37s{color} | {color:blue}
Used deprecated FindBugs config; considering switching to SpotBugs. {color} |
| {color:red}-1{color} | {color:red} findbugs {color} | {color:red}  0m 25s{color} | {color:red}
tez-tfile-parser in master failed. {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m  9s{color} | {color:blue}
Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  1m 54s{color}
| {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m 38s{color} |
{color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  1m 38s{color} | {color:green}
the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 14s{color}
| {color:green} The patch passed checkstyle in tez-api {color} |
| {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange}  0m 11s{color}
| {color:orange} tez-runtime-internals: The patch generated 1 new + 69 unchanged - 3 fixed
= 70 total (was 72) {color} |
| {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange}  0m 19s{color}
| {color:orange} tez-runtime-library: The patch generated 2 new + 285 unchanged - 1 fixed
= 287 total (was 286) {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 15s{color}
| {color:green} The patch passed checkstyle in tez-mapreduce {color} |
| {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange}  0m 42s{color}
| {color:orange} tez-dag: The patch generated 12 new + 1120 unchanged - 10 fixed = 1132 total
(was 1130) {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 11s{color}
| {color:green} tez-plugins/tez-aux-services: The patch generated 0 new + 47 unchanged - 1
fixed = 47 total (was 48) {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m  8s{color}
| {color:green} The patch passed checkstyle in tez-tfile-parser {color} |
| {color:red}-1{color} | {color:red} whitespace {color} | {color:red}  0m  0s{color} | {color:red}
The patch has 1 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>.
Refer https://git-scm.com/docs/git-apply {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m 50s{color} |
{color:green} the patch passed {color} |
| {color:red}-1{color} | {color:red} findbugs {color} | {color:red}  0m 11s{color} | {color:red}
tez-tfile-parser in the patch failed. {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  1m 54s{color} | {color:green}
tez-api in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 30s{color} | {color:green}
tez-runtime-internals in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  5m  7s{color} | {color:green}
tez-runtime-library in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  1m  8s{color} | {color:green}
tez-mapreduce in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  4m  9s{color} | {color:green}
tez-dag in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 33s{color} | {color:green}
tez-aux-services in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 10s{color} | {color:green}
tez-tfile-parser in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  1m  9s{color}
| {color:green} The patch does not generate ASF License warnings. {color} |
| {color:black}{color} | {color:black} {color} | {color:black} 72m  0s{color} | {color:black}
{color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-87/1/artifact/out/Dockerfile
|
| GITHUB PR | https://github.com/apache/tez/pull/87 |
| JIRA Issue | TEZ-4095 |
| Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile
|
| uname | Linux 362b035d2304 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 / fb81c1158 |
| Default Java | Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01 |
| findbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-87/1/artifact/out/branch-findbugs-tez-tools_tez-tfile-parser.txt
|
| checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-87/1/artifact/out/diff-checkstyle-tez-runtime-internals.txt
|
| checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-87/1/artifact/out/diff-checkstyle-tez-runtime-library.txt
|
| checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-87/1/artifact/out/diff-checkstyle-tez-dag.txt
|
| whitespace | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-87/1/artifact/out/whitespace-eol.txt
|
| findbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-87/1/artifact/out/patch-findbugs-tez-tools_tez-tfile-parser.txt
|
|  Test Results | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-87/1/testReport/
|
| Max. process+thread count | 1656 (vs. ulimit of 5500) |
| modules | C: tez-api tez-runtime-internals tez-runtime-library tez-mapreduce tez-dag tez-plugins/tez-aux-services
tez-tools/tez-tfile-parser U: . |
| Console output | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-87/1/console |
| versions | git=2.17.1 maven=3.6.0 findbugs=3.0.1 |
| Powered by | Apache Yetus 0.12.0 https://yetus.apache.org |


This message was automatically generated.



> Review of Debug Logging
> -----------------------
>
>                 Key: TEZ-4095
>                 URL: https://issues.apache.org/jira/browse/TEZ-4095
>             Project: Apache Tez
>          Issue Type: Improvement
>            Reporter: David Mollitor
>            Priority: Minor
>         Attachments: TEZ-4095.1.patch, TEZ-4095.2.patch, TEZ-4095.2.patch
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> # Remove superfluous debug logging guards
> # Create parameterized logging statements where appropriate



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Mime
View raw message