-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
App Update #24198
Comments
The background jobs that would be defined in info.xml would be seperate from the job queue (job queue is for one-off async work) |
Just to be sure - we only have one queue and there is nothing going to change with this respect thinking about the executor mechanism you are building for 9.1 So no matter if the jobs are one shot of cron like they live in the same queue and will be equally executed amongst all job executors. So as soon as a background job is defined in info.xml - we should throw it onto the job queue. |
Background jobs (currently defined in update/install.php, to be moved to info.xml) execution will stay the same (cron.php) and is meant for periodic background jobs (news app sync, syncing of remote system address books, etc) Job queue will get it's new fancy executor for 9.1 and is meant for one-off async work that is triggered by some action (trashbin expiry) |
Hmm .. I see no need to separate them in terms of who is executing them. |
Mainly done - remaining tasks can be ignored for now -> close |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
ToDos:
The text was updated successfully, but these errors were encountered: