[
https://issues.apache.org/jira/browse/YARN-3998?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15214532#comment-15214532
]
Jun Gong commented on YARN-3998:
--------------------------------
Very sorry for late. I just a attached a new patch 07.patch.
In the new patch:
1. It treats RESTART in a first class manner: add new state RELAUNCHING for
container, and ContainerRelaunch.
2. ContainerLaunch records and stores container's working directory and log
directory, ContainerRelaunch will reuse previous working directory, log
directory and container's launch script. If previous directories/files are not
available, it will fail to relaunch. We could change this behavior afterwards.
3. Address [~vvasudev]'s all comments to 06.patch.
> Add retry-times to let NM re-launch container when it fails to run
> ------------------------------------------------------------------
>
> Key: YARN-3998
> URL: https://issues.apache.org/jira/browse/YARN-3998
> Project: Hadoop YARN
> Issue Type: Sub-task
> Reporter: Jun Gong
> Assignee: Jun Gong
> Attachments: YARN-3998.01.patch, YARN-3998.02.patch,
> YARN-3998.03.patch, YARN-3998.04.patch, YARN-3998.05.patch,
> YARN-3998.06.patch, YARN-3998.07.patch
>
>
> I'd like to add a field(retry-times) in ContainerLaunchContext. When AM
> launches containers, it could specify the value. Then NM will re-launch the
> container 'retry-times' times when it fails to run(e.g.exit code is not 0).
> It will save a lot of time. It avoids container localization. RM does not
> need to re-schedule the container. And local files in container's working
> directory will be left for re-use.(If container have downloaded some big
> files, it does not need to re-download them when running again.)
> We find it is useful in systems like Storm.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
