Status stay in retrying

[jamalyan]

Hi there. I am trying to use this great package at first time.
But I got one issue.
When job get exception, the status not changing to 'failed', but stay 'retrying'.
In DefaultEventManager I have changed exceptionOccurred method functionality to get what I need.

'status' => ($event->job->attempts() == $event->job->maxTries()) ? $this->getEntity()::STATUS_FAILED : $this->getEntity()::STATUS_RETRYING,

[sstankov]

Just we verify we experienced the same one.

[o15a3d4l11s2]

Despite the fix I am still getting this issue - if the job fails, status stay as 'retrying', although the job is moved to the failed_jobs table by Laravel and there are no more retries. I think this is also related to #63.

[mylemans]

Despite the fix I am still getting this issue - if the job fails, status stay as 'retrying', although the job is moved to the failed_jobs table by Laravel and there are no more retries. I think this is also related to #63.

Same issue here. Any update on this?

To add, this gets logged in the failed jobs table:

Illuminate\Queue\MaxAttemptsExceededException: Modules\Platform\Jobs\XXX\XXXJob has been attempted too many times or run too long. The job may have previously timed out.

'Causer' of the exception is using the WithoutOverlapping middleware

[JuanS]

Having the same problem here, made a PR #72

[mylemans]

@imTigger Any idea when you'll publish a new release with this fix?

[MathiasWeisheit]

@mylemans what can we do do get the change in a new release?

[mylemans]

@mylemans what can we do do get the change in a new release?

Why are you asking me? ;D

[MathiasWeisheit]

That's right!
I already searched for forks with more engagement.