MAPREDUCE-6826. Job fails with InvalidStateTransitonException: Invalid event: JOB_TASK_COMPLETED at SUCCEEDED/COMMITTING. Contributed by Bilwa S T.
This commit is contained in:
parent
ce4ec74453
commit
d4e36409d4
|
@ -422,7 +422,8 @@ public class JobImpl implements org.apache.hadoop.mapreduce.v2.app.job.Job,
|
|||
EnumSet.of(JobEventType.JOB_UPDATED_NODES,
|
||||
JobEventType.JOB_TASK_ATTEMPT_FETCH_FAILURE,
|
||||
JobEventType.JOB_TASK_ATTEMPT_COMPLETED,
|
||||
JobEventType.JOB_MAP_TASK_RESCHEDULED))
|
||||
JobEventType.JOB_MAP_TASK_RESCHEDULED,
|
||||
JobEventType.JOB_TASK_COMPLETED))
|
||||
|
||||
// Transitions from SUCCEEDED state
|
||||
.addTransition(JobStateInternal.SUCCEEDED, JobStateInternal.SUCCEEDED,
|
||||
|
@ -441,7 +442,8 @@ public class JobImpl implements org.apache.hadoop.mapreduce.v2.app.job.Job,
|
|||
JobEventType.JOB_TASK_ATTEMPT_FETCH_FAILURE,
|
||||
JobEventType.JOB_AM_REBOOT,
|
||||
JobEventType.JOB_TASK_ATTEMPT_COMPLETED,
|
||||
JobEventType.JOB_MAP_TASK_RESCHEDULED))
|
||||
JobEventType.JOB_MAP_TASK_RESCHEDULED,
|
||||
JobEventType.JOB_TASK_COMPLETED))
|
||||
|
||||
// Transitions from FAIL_WAIT state
|
||||
.addTransition(JobStateInternal.FAIL_WAIT,
|
||||
|
|
|
@ -203,7 +203,7 @@ public class TestJobImpl {
|
|||
public void testCheckJobCompleteSuccess() throws Exception {
|
||||
Configuration conf = new Configuration();
|
||||
conf.set(MRJobConfig.MR_AM_STAGING_DIR, stagingDir);
|
||||
AsyncDispatcher dispatcher = new AsyncDispatcher();
|
||||
DrainDispatcher dispatcher = new DrainDispatcher();
|
||||
dispatcher.init(conf);
|
||||
dispatcher.start();
|
||||
CyclicBarrier syncBarrier = new CyclicBarrier(2);
|
||||
|
@ -225,6 +225,11 @@ public class TestJobImpl {
|
|||
JobEventType.JOB_MAP_TASK_RESCHEDULED));
|
||||
assertJobState(job, JobStateInternal.COMMITTING);
|
||||
|
||||
job.handle(new JobEvent(job.getID(),
|
||||
JobEventType.JOB_TASK_COMPLETED));
|
||||
dispatcher.await();
|
||||
assertJobState(job, JobStateInternal.COMMITTING);
|
||||
|
||||
// let the committer complete and verify the job succeeds
|
||||
syncBarrier.await();
|
||||
assertJobState(job, JobStateInternal.SUCCEEDED);
|
||||
|
@ -236,6 +241,11 @@ public class TestJobImpl {
|
|||
job.handle(new JobEvent(job.getID(),
|
||||
JobEventType.JOB_MAP_TASK_RESCHEDULED));
|
||||
assertJobState(job, JobStateInternal.SUCCEEDED);
|
||||
|
||||
job.handle(new JobEvent(job.getID(),
|
||||
JobEventType.JOB_TASK_COMPLETED));
|
||||
dispatcher.await();
|
||||
assertJobState(job, JobStateInternal.SUCCEEDED);
|
||||
|
||||
dispatcher.stop();
|
||||
commitHandler.stop();
|
||||
|
|
Loading…
Reference in New Issue