hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jian He (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-3584) [Log mesage correction] : MIssing space in Diagnostics message
Date Wed, 06 May 2015 22:23:00 GMT

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

Jian He commented on YARN-3584:
-------------------------------

thanks [~nijel] for the patch.  to make it more clear, could you flip below order ? append
the "Diagnostics" first, then append "For more detailed output..."

{code}
    if (this.getTrackingUrl() != null) {
      diagnosticsBuilder.append("For more detailed output,").append(
        " check application tracking page: ").append(
        this.getTrackingUrl()).append(
        " Then, click on links to logs of each attempt.\n");
    }
    diagnosticsBuilder.append("Diagnostics: ").append(status.getDiagnostics())
        .append("Failing this attempt");
    return diagnosticsBuilder.toString();
{code}

> [Log mesage correction] : MIssing space in Diagnostics message
> --------------------------------------------------------------
>
>                 Key: YARN-3584
>                 URL: https://issues.apache.org/jira/browse/YARN-3584
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: nijel
>            Assignee: nijel
>            Priority: Trivial
>              Labels: newbie
>         Attachments: YARN-3584-1.patch
>
>
> For more detailed output, check application tracking page: https://szxciitslx17640:26001/cluster/app/application_1430810985970_0020{color:red}Then{color},
click on links to logs of each attempt.
> In this Then is not part of thr URL. Better to use a space in between so that the URL
can be copied directly for analysis



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message