[
https://issues.apache.org/jira/browse/YARN-5561?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15554926#comment-15554926
]
Rohith Sharma K S commented on YARN-5561:
-----------------------------------------
bq We do not store configurations at container and app attempt level. So we can
leave out query params associated with it,
bq. Similarly no metrics are stored at app attempt level. Same goes for
relationships. We can add them when we support them, if not for anything but
just to reduce lines in code and documentation
That said, these REST end points are enhancement of getEntitties and getEntity.
So, do not want to change any of the query params associated with app-attempts
and container. In future, we never know what metrics/configs/relationships
would be stored in attempts/containers. Lets keep it as it, if user query
configs/metrics then he gets result in empty. It indicates , there is no
metrics or configs published.
bq. Javadoc will have to be updated I think otherwise we will get a -1 in QA
report. Yes this will unnecessarily increase code size but then there is no
other way out.
This is one of point of concern for code readability. May be we can capture in
document, but not in java class file. Thought?
bq. Update tests for app attempts too ?
Let me update the tests and will update new patch.
> [Atsv2] : Support for ability to retrieve apps/app-attempt/containers and
> entities via REST
> -------------------------------------------------------------------------------------------
>
> Key: YARN-5561
> URL: https://issues.apache.org/jira/browse/YARN-5561
> Project: Hadoop YARN
> Issue Type: Sub-task
> Components: timelinereader
> Reporter: Rohith Sharma K S
> Assignee: Rohith Sharma K S
> Attachments: 0001-YARN-5561.YARN-5355.patch, YARN-5561.02.patch,
> YARN-5561.patch, YARN-5561.v0.patch
>
>
> ATSv2 model lacks retrieval of {{list-of-all-apps}},
> {{list-of-all-app-attempts}} and {{list-of-all-containers-per-attempt}} via
> REST API's. And also it is required to know about all the entities in an
> applications.
> It is pretty much highly required these URLs for Web UI.
> New REST URL would be
> # GET {{/ws/v2/timeline/apps}}
> # GET {{/ws/v2/timeline/apps/\{app-id\}/appattempts}}.
> # GET
> {{/ws/v2/timeline/apps/\{app-id\}/appattempts/\{attempt-id\}/containers}}
> # GET {{/ws/v2/timeline/apps/\{app id\}/entities}} should display list of
> entities that can be queried.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
