-
Notifications
You must be signed in to change notification settings - Fork 66
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
Feature request: Immediately start a job #71
Comments
I'm probably missing something but if no queue features are needed, why not call the function directly? |
There are several reasons why I've chosen to go this route. First, you get a simpler codebase by structuring your application to use the provided events. This keeps users from rewriting code for different path handling. In my case, I have an Furthermore, it's a much simpler codebase by being able to write code to use job info like IDs and such. Edit:
Several queue features are still in use here. Namely, being able to have timeouts, events, job information, etc. The only feature not being used is concurrency limit. We want most jobs to respect concurrency limits (an example is bulk processing) whilst other jobs may not be resource intensive thus can run immediately. One workaround is to use two instances of Queue however this has the disadvantage of having duplicate code and multiple code paths. Please see pull request #73 for an example. The work is also already provided for you with minimal changes. Does this answer your question? Willing to provide some concrete examples. |
Looking to be able to able to start a job immediately, skipping concurrency and other limits.
The text was updated successfully, but these errors were encountered: