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

Konstantinos Karanasos commented on YARN-6059:
----------------------------------------------

Thanks Hitesh. Regarding your questions:
bq. We don't launch the paused container and instead we simply kill it.
bq. Hmm..since we kill the container an event for that will be raised 
automatically. Am I missing something?
I understand that you immediately send a kill signal to the process, which is 
good.
This is sufficient for killing the actual container and reclaiming the 
resources in the NM, but it does not let the RM know what happened to the 
container. Check, for example, the {{ContainerRelaunch}} or the 
{{RecoveredContainerLaunch}} classes. Once the container has exited, they send 
a {{ContainerExitEvent}}. We should send a similar event once the killing of 
the paused container is done, setting the right exit code.

bq. In RecoveredContainerLaunch, indentation needs to be fixed.
My bad, that was a typo -- I meant {{ContainersLauncher}}: we need to fix the 
indentation of the case statement.

> Update paused container state in the state store
> ------------------------------------------------
>
>                 Key: YARN-6059
>                 URL: https://issues.apache.org/jira/browse/YARN-6059
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Hitesh Sharma
>            Assignee: Hitesh Sharma
>         Attachments: YARN-5216-YARN-6059.001.patch, 
> YARN-6059-YARN-5972.001.patch, YARN-6059-YARN-5972.002.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to