[ 
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)

Reply via email to