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.

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

[5.1] BeanstalkdJob bury function delete job unexpectedly #13963

Merged
merged 2 commits into from
Jun 16, 2016

Conversation

joeyew
Copy link
Contributor

@joeyew joeyew commented Jun 13, 2016

Job will be deleted whenever bury job function is called.

Due to the release flag is not mark, every job goin to bury will be delete eventually.

        if (! $job->isDeletedOrReleased()) {
            $job->delete();
        }

Job will be deleted whenever bury job function is called.

Due to the release flag is not mark, every job goin to bury will be delete eventually.
```php
        if (! $job->isDeletedOrReleased()) {
            $job->delete();
        }
```
@GrahamCampbell GrahamCampbell changed the title [BUG][5.1] BeanstalkdJob bury function delete job unexpectedly [5.1] BeanstalkdJob bury function delete job unexpectedly Jun 13, 2016
@taylorotwell
Copy link
Member

Can find no indication that releasing is needed before burying. Please send link to documentation that states this is necessary.

@joeyew
Copy link
Contributor Author

joeyew commented Jun 14, 2016

Yes. No releasing is needed, it's just a workaround so that $release set to true and the job won't be deleted.

The bug is the job would never be bury, it's just deleted at last.

@taylorotwell taylorotwell reopened this Jun 14, 2016
@taylorotwell
Copy link
Member

I'm not sure releasing it is the answer then. We should solve the problem in another way.

@joeyew
Copy link
Contributor Author

joeyew commented Jun 14, 2016

Yes. Thanks Taylor. it's affected 5.2 as well.

@joeyew
Copy link
Contributor Author

joeyew commented Jun 14, 2016

Should we add a bury and isBuried function in Job class? But there is only beanstalkd got this bury function

@taylorotwell taylorotwell merged commit d4afdd2 into laravel:5.1 Jun 16, 2016
@joeyew joeyew deleted the patch-1 branch June 16, 2016 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants