[ 
https://issues.apache.org/jira/browse/YARN-1700?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13912551#comment-13912551
 ] 

Zhijie Shen commented on YARN-1700:
-----------------------------------

bq. Log URL is physically nullable, but in the current source code as is it 
never changes after the launch and is not null.

Good catch. This is another issue of the current code, which should be fixed in 
YARN-1685 as well. See my prior comment in YARN-1413: 
https://issues.apache.org/jira/browse/YARN-1413?focusedCommentId=13866844&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13866844

When the container is running, the log url should point to the NM web page, 
which serves the running container log; when it is finished, the log url should 
then be updated (See TODO in RMContainerImpl.java), and point to the AHS web 
page, which serves the aggregated log. 

> AHS records non-launched containers
> -----------------------------------
>
>                 Key: YARN-1700
>                 URL: https://issues.apache.org/jira/browse/YARN-1700
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>    Affects Versions: 2.4.0
>            Reporter: Gera Shegalov
>            Assignee: Gera Shegalov
>         Attachments: YARN-1700.v01.patch, YARN-1700.v02.patch
>
>
> When testing AHS with a MR sleep job, AHS sometimes threw NPE out  of 
> AppAttemptBlock.render because logUrl in container report was null. I 
> realized that this is because AHS may record containers that never launch.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to