[
https://issues.apache.org/jira/browse/TEZ-2097?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14731015#comment-14731015
]
Hitesh Shah commented on TEZ-2097:
----------------------------------
Comments:
- "atsEntity.addOtherInfo(ATSConstants.IN_PROGRESS_LOGS_URL,
event.getContainerLogs());" will override the value in ATS. Shouldn't this be
effectively an append in some sense? Given that with ATS, we will need to
effectively do a getAndSet(), may be use "ATSConstants.IN_PROGRESS_LOGS_URL +
appAttemptId suffix" to insert multiple container log urls? The UI can then
find all fields under otherInfo that match the inprogress prefix?
- Unit tests have the ctor changed but should be also test whether the value
is being used correctly.
> TEZ-UI Add dag logs
> -------------------
>
> Key: TEZ-2097
> URL: https://issues.apache.org/jira/browse/TEZ-2097
> Project: Apache Tez
> Issue Type: Bug
> Components: UI
> Reporter: Jeff Zhang
> Assignee: Jonathan Eagles
> Priority: Critical
> Attachments: TEZ-2097.1.patch, TEZ-2097.2.patch
>
>
> If dag fails due to AM error, there's no way to check the dag logs on tez-ui.
> Users have to grab the app logs.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
