Closed (won't fix)
Project:
Hosting
Version:
5.x-0.1-rc1
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
29 Jan 2009 at 21:49 UTC
Updated:
17 Aug 2010 at 21:55 UTC
Jump to comment: Most recent file
Comments
Comment #1
anarcat commentedComment #2
anarcat commentedBasically, I think the current design of the cron queue is wrong: we shouldn't run cron jobs in parallel unless we really have to. Also, the control should be over the frequency of the cron jobs (which could be per site) than the number of cron jobs per run. When a cron job is due, it's due, that's it, it needs to be ran. The frequency should just be obeyed.
To alleviate the load on the server (which is an issue we are witnessing now, see #606072: Cron should be disabled for disabled sites), that frequency can be changed by the administrator, or in the case of per site cron frequencies, a minimum can be enforced.
Comment #3
adrian commentedignore that block. i want to remove it in any case.
that block shows the CALCULATED amount of cron jobs that will be started at the CALCULATED frequency.
it is configured by frequency. see the screenshot, you are telling it to run cron on all X sites every Y
it then splits the site crons into equal parts over the specified time period, and it has a flexibility that it starts doing every 10 minutes, but once it reaches a threshold of Z sites per run , it spawns more frequent processes.
I still think we should run them in paralel, because we don't need any output from the script, and holding up the spawning of more cron jobs for the previous ones to finish means we will run into situations where the previous batch hasn't finished when the new one starts.
Comment #4
adrian commentedscreenshot
Comment #5
adrian commentedClosing this , even though it's not really a duplicate it's the exact opposite of
#695244: hosting-cron killed my server