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

Bikas Saha commented on YARN-498:
---------------------------------

I had originally left the loop without a timeout because eventually YARN is 
supposed to recognize that the app succeeded/killed/failed or it would expire 
and fail the app. Is that not happening?

I agree that a timeout would be more useful. How about adding a timeout to the 
monitoring method such that it throws an exception if the desired end state is 
not reached within that time. With the code change in the patch, it would end 
up crashing with an NPE, right?
                
> Unmanaged AM launcher does not set various constants in env for an AM, also 
> does not handle failed AMs properly
> ---------------------------------------------------------------------------------------------------------------
>
>                 Key: YARN-498
>                 URL: https://issues.apache.org/jira/browse/YARN-498
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Hitesh Shah
>            Assignee: Hitesh Shah
>         Attachments: YARN-498.1.patch, YARN-498.2.patch, YARN-498.wip.patch
>
>
> Currently, it only sets the app attempt id which is really not required as 
> AMs are only expected to extract it from the container id.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to