You are viewing a plain text version of this content. The canonical link for it is here.

Posted to yarn-issues@hadoop.apache.org by "Arun Suresh (JIRA)" <ji...@apache.org> on 2015/09/08 20:30:47 UTC

[jira] [Commented] (YARN-4086) Allow Aggregated Log readers to handle HAR files

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

Arun Suresh commented on YARN-4086:
-----------------------------------

Thanks for the patch Robert.

It Looks good save a minor nit :
* In the 'testFetchApplictionLogsHar', when asserting the value of contents of the sysoutStream, you may want to just check if the output contains some impartant/relevant strings rather than matching the whole output, else the testcase would end up quite brittle, and requiring constant changes (especially if the output format changes)

+1 post jenkins

> Allow Aggregated Log readers to handle HAR files
> ------------------------------------------------
>
>                 Key: YARN-4086
>                 URL: https://issues.apache.org/jira/browse/YARN-4086
>             Project: Hadoop YARN
>          Issue Type: Improvement
>    Affects Versions: 2.8.0
>            Reporter: Robert Kanter
>            Assignee: Robert Kanter
>         Attachments: YARN-4086.001.patch
>
>
> This is for the YARN changes for MAPREDUCE-6415.  It allows the yarn CLI and web UIs to read aggregated logs from HAR files.



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