w33ble/elastiq
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

emit job fail events, don't throw

Browse Source
This commit is contained in:
Joe Fleming
2016-05-03 11:35:09 -07:00
parent 552abb7ddd
commit f725824c5f
1 changed files with 3 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/worker.js
View File

@@ -159,16 +159,13 @@ export default class Job extends events.EventEmitter {
// job execution failed // job execution failed
if (jobErr.type === 'WorkerTimeout') { if (jobErr.type === 'WorkerTimeout') {
this.debug(`Timeout on job ${job._id}`); this.debug(`Timeout on job ${job._id}`);
this.emit('job_timeout', jobErr);
return; return;
} }
this.debug(`Failure occurred on job ${job._id}`); this.debug(`Failure occurred on job ${job._id}`);
return this._failJob(job, jobErr.toString()) this.emit('job_error', jobErr);
.catch(() => false) return this._failJob(job, jobErr.toString());
.then(() => {
this.emit('job_error', jobErr);
throw jobErr;
});
}); });
} }