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

Mayank Bansal commented on YARN-2320:
-------------------------------------

I think  overal looks ok however Have to run.

some small comments

shouldn't we use N/A in convertToApplicationAttemptReport instead of null ?
Similarly for convertToApplicationReport?
Similary for convertToContainerReport?

> Removing old application history store after we store the history data to 
> timeline store
> ----------------------------------------------------------------------------------------
>
>                 Key: YARN-2320
>                 URL: https://issues.apache.org/jira/browse/YARN-2320
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: timelineserver
>            Reporter: Zhijie Shen
>            Assignee: Zhijie Shen
>         Attachments: YARN-2320.1.patch, YARN-2320.2.patch
>
>
> After YARN-2033, we should deprecate application history store set. There's 
> no need to maintain two sets of store interfaces. In addition, we should 
> conclude the outstanding jira's under YARN-321 about the application history 
> store.



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

Reply via email to