Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[5.3] Check attempts before firing queue job. #15319

Merged
merged 4 commits into from
Sep 7, 2016
Merged

[5.3] Check attempts before firing queue job. #15319

merged 4 commits into from
Sep 7, 2016

Commits on Sep 7, 2016

  1. Check jobs before working to see if they have already been received t… 

    …oo many times.

Resolves an issue with the `--timeout` feature where jobs the repeatedly timed out would never be marked as `failed`, as the worker process would be killed before
it could reach the failing logic.

To maintain compatibility there are now two checks against the number of attempts a job has had, one before working the job and one in the case of an job raising an exception.

see laravel#15317 for more details.
    Max Brokman committed Sep 7, 2016
    1 Configuration menu
    Copy the full SHA
    de2e30a View commit details
    Browse the repository at this point in the history

  2. Style fixes for de2e30a

    Max Brokman committed Sep 7, 2016
    Configuration menu
    Copy the full SHA
    7520913 View commit details
    Browse the repository at this point in the history

  3. Style fixes suggested in laravel#15319

    Max Brokman committed Sep 7, 2016
    Configuration menu
    Copy the full SHA
    73752e0 View commit details
    Browse the repository at this point in the history

  4. Remove extra line

    Max Brokman committed Sep 7, 2016
    Configuration menu
    Copy the full SHA
    54d2c6b View commit details
    Browse the repository at this point in the history