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 

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

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

Reply via email to