[
https://issues.apache.org/jira/browse/YARN-1884?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14355392#comment-14355392
]
Zhijie Shen commented on YARN-1884:
-----------------------------------
Looks good to me overall. In the test case, can we change the test string from
host:port to http://host:port. It's not because the current string is not
right to pass the test, but I hope it could be more representative of a real
http node address.
> ContainerReport should have nodeHttpAddress
> -------------------------------------------
>
> Key: YARN-1884
> URL: https://issues.apache.org/jira/browse/YARN-1884
> Project: Hadoop YARN
> Issue Type: Sub-task
> Reporter: Zhijie Shen
> Assignee: Xuan Gong
> Attachments: YARN-1884.1.patch, YARN-1884.2.patch
>
>
> In web UI, we're going to show the node, which used to be to link to the NM
> web page. However, on AHS web UI, and RM web UI after YARN-1809, the node
> field has to be set to nodeID where the container is allocated. We need to
> add nodeHttpAddress to the containerReport to link users to NM web page
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
