[
https://issues.apache.org/jira/browse/YARN-987?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13815351#comment-13815351
]
Zhijie Shen commented on YARN-987:
----------------------------------
No need to do this change, which will break TestMemoryApplicationHistoryStore
and TestFileSystemApplicationHistoryStore.
{code}
- appAttemptId, appAttemptId.toString(), 0,
+ appAttemptId, "localhost", 0,
{code}
Why not simply changing
{code}
+ Assert.assertEquals("localhost", appReport.getHost());
{code}
to
{code}
+ Assert.assertEquals(appAttemptId.toString(), appReport.getHost());
{code}
> Adding History Service to use Store and converting Historydata to Report
> ------------------------------------------------------------------------
>
> Key: YARN-987
> URL: https://issues.apache.org/jira/browse/YARN-987
> Project: Hadoop YARN
> Issue Type: Sub-task
> Reporter: Mayank Bansal
> Assignee: Mayank Bansal
> Attachments: YARN-987-1.patch, YARN-987-2.patch, YARN-987-3.patch,
> YARN-987-4.patch, YARN-987-5.patch, YARN-987-6.patch, YARN-987-7.patch
>
>
--
This message was sent by Atlassian JIRA
(v6.1#6144)
