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

Disable sleep while queue items available #1250

Merged

Conversation

mordilion
Copy link
Contributor

This change is disabling the Subscription-Polling-Sleep as long there are items in queues. If there are no items in queues, then the gathering of items is not constantly penetrating the Database.

We are using it already in our production environment and it works perfectly fine without any Database issues.

@jprangenbergde
Copy link

I like!

@makasim
Copy link
Member

makasim commented May 17, 2022

I've hesitated with merging this because I am not sure I can spot all the possible scenario and bugs it might introduce.

On the other hand, I dont want to be a blocker for improvements.

I can merge if you are sure it works well.

Please tell.

@mordilion
Copy link
Contributor Author

@makasim we are using it in a productive environment already since the 10th January 2022 and we have daily between 200.000 and 300.000 items in ~40 queues.

Based on that I am 100% sure that it works! :)

@makasim makasim merged commit 8b8f2f8 into php-enqueue:master May 17, 2022
@makasim
Copy link
Member

makasim commented May 17, 2022

@mordilion thanks!

@makasim
Copy link
Member

makasim commented May 17, 2022

@mordilion
Copy link
Contributor Author

@makasim thanks for merging it :)

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.

3 participants