MAPREDUCE-6826. Job fails with InvalidStateTransitonException: Invalid event: JOB_TASK_COMPLETED at SUCCEEDED/COMMITTING. Contributed by Bilwa S T.
(cherry picked from commit d4e36409d4
)
This commit is contained in:
parent
ae2d856515
commit
c70f5eb8fa
@ -422,7 +422,8 @@ JobEventType.JOB_KILL, new KillTasksTransition())
|
||||
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 @@ JobEventType.JOB_KILL, new KillTasksTransition())
|
||||
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,
|
||||
|
@ -204,7 +204,7 @@ public void testCommitJobFailsJob() throws Exception {
|
||||
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);
|
||||
@ -226,6 +226,11 @@ public void testCheckJobCompleteSuccess() throws Exception {
|
||||
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);
|
||||
@ -237,6 +242,11 @@ public void testCheckJobCompleteSuccess() throws Exception {
|
||||
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
Block a user