[
https://issues.apache.org/jira/browse/YARN-8517?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16546990#comment-16546990
]
Szilard Nemeth commented on YARN-8517:
--------------------------------------
Hi [~bsteinbach]!
Thanks for the patch!
Couple of comments:
1. Containers for an application attempt API --> Please change to "Containers
for an Application Attempt API" (use uppercase A letters)
2. Please change "The amount of virtual cores allocated the container" to "The
amount of virtual cores allocated *for* the container"
3. Please change "The elapsed in ms since the startedTime" to "The elapsed
*time* in ms since the startedTime"
4. Did you intentionally exclude "diagnosticsInfo" from the field table of
ContainerInfo?
5. Please change "Final exit status for the container" to "Final exit status
*of* the container"
6. The URLs are wrong in the HTTP request section for both introduced API
methods (both for JSON and XML): "GET
http://rm-http-address:port/ws/v1/cluster/nodes"; is wrong
> getContainer and getContainers ResourceManager REST API methods are not
> documented
> ----------------------------------------------------------------------------------
>
> Key: YARN-8517
> URL: https://issues.apache.org/jira/browse/YARN-8517
> Project: Hadoop YARN
> Issue Type: Improvement
> Components: resourcemanager
> Reporter: Szilard Nemeth
> Assignee: Antal Bálint Steinbach
> Priority: Major
> Labels: newbie, newbie++
> Attachments: YARN-8517.001.patch
>
>
> Looking at the documentation here:
> https://hadoop.apache.org/docs/r3.1.0/hadoop-yarn/hadoop-yarn-site/ResourceManagerRest.html
> I cannot find documentation for 2 RM REST endpoints:
> - /apps/\{appid\}/appattempts/\{appattemptid\}/containers
> - /apps/\{appid\}/appattempts/\{appattemptid\}/containers/\{containerid\}
> I suppose they are not intentionally undocumented.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
