[jira] [Commented] (YARN-4029) Update LogAggregationStatus to store on finish

["Rohith Sharma K S (JIRA)" <ji...@apache.org>]

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

Rohith Sharma K S commented on YARN-4029:
-----------------------------------------

Thanks [~bibinchundatt] for providing patch, overall patch looks good.
Some nits
# LogAggStatusProto values can have without *LOGAGG_* so that PBImpl need not have logic to add prefix and replace it.

> Update LogAggregationStatus to store on finish
> ----------------------------------------------
>
>                 Key: YARN-4029
>                 URL: https://issues.apache.org/jira/browse/YARN-4029
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Bibin A Chundatt
>            Assignee: Bibin A Chundatt
>         Attachments: 0001-YARN-4029.patch, 0002-YARN-4029.patch, 0003-YARN-4029.patch, Image.jpg
>
>
> Currently the log aggregation status is not getting updated to Store. When RM is restarted will show NOT_START. 
> Steps to reproduce
> ================
> 1.Submit mapreduce application
> 2.Wait for completion
> 3.Once application is completed switch RM
> *Log Aggregation Status* are changing
> *Log Aggregation Status* from SUCCESS to NOT_START



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