[
https://issues.apache.org/jira/browse/YARN-5286?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15354717#comment-15354717
]
Varun Saxena commented on YARN-5286:
------------------------------------
[~hex108], thanks for the patch.
A couple of comments :
AM RPC address in AppInfo.java is being filled inside the check for master
container not being null.
But Master container is set when its allocated by scheduler, whereas AM host
and RPC port is set when AM registers. There will be some time difference
between both these events. I think we can explicitly check if host is not "N/A"
and RPC port is not -1 to fill AM RPC address in response.
Moreover, the additional field added can be added in documentation too in
{{hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md}}. That is add the
field in the response under the apps and app REST endpoint sections.
> Add RPC port info in RM web service's response when getting app status
> ----------------------------------------------------------------------
>
> Key: YARN-5286
> URL: https://issues.apache.org/jira/browse/YARN-5286
> Project: Hadoop YARN
> Issue Type: Bug
> Reporter: Jun Gong
> Assignee: Jun Gong
> Attachments: YARN-5286.01.patch, YARN-5286.02.patch
>
>
> When getting app status by RM web service({{/ws/v1/cluster/apps/\{appid\}}}),
> there is no RPC port info in the response. The port info is very important to
> communicate with AM.
> BTW: there is RPC port info when running {{bin/yarn application -status
> appid}}.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
