[
https://issues.apache.org/jira/browse/MAPREDUCE-5086?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13616496#comment-13616496
]
Bikas Saha commented on MAPREDUCE-5086:
---------------------------------------
Can we please make sure that the next patch does not have any unnecessary white
space.
Why is this change needed?
{code}
@@ -79,16 +79,72 @@ public void testDeletionofStaging() throws IOException {
when(fs.exists(stagingDir)).thenReturn(true);
ApplicationAttemptId attemptId = recordFactory.newRecordInstance(
ApplicationAttemptId.class);
- attemptId.setAttemptId(0);
+ attemptId.setAttemptId(1);
{code}
[~jlowe]Do you have further comments? The JobImpl cast is needed because
internal state is not something thats exposed via the Job interface. I am fine
with changing the interface except for the fact that it exposes the state
machine concept that is internal to JobImpl.
> MR app master deletes staging dir when sent a reboot command from the RM
> ------------------------------------------------------------------------
>
> Key: MAPREDUCE-5086
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-5086
> Project: Hadoop Map/Reduce
> Issue Type: Bug
> Reporter: jian he
> Assignee: jian he
> Attachments: YARN-472.1.patch, YARN-472.2.patch, YARN-472.3.patch,
> YARN-472.4.patch, YARN-472.5.patch, YARN-472.6.patch
>
>
> If the RM is restarted when the MR job is running, then it sends a reboot
> command to the job. The job ends up deleting the staging dir and that causes
> the next attempt to fail.
--
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
