mahara-contributors team mailing list archive
-
mahara-contributors team
-
Mailing list archive
-
Message #06871
[Bug 891170] Re: race conditions in cron
Thanks for that Andrew and Tim.
I reckon we should target this for 1.5 just because it's the kind of bug
that's really painful to debug when it happens in the wild.
** Changed in: mahara
Status: New => Triaged
** Changed in: mahara
Milestone: None => 1.5.0
** Changed in: mahara
Importance: Low => Medium
--
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
https://bugs.launchpad.net/bugs/891170
Title:
race conditions in cron
Status in Mahara ePortfolio:
Triaged
Bug description:
Thanks to tim hunt at the OU for pointing the first issue out:
It's possible for cron_lock to fail to get the $started value if the row has been deleted between attempting to insert the record, and attempting to retrieve the start date
http://gitorious.org/mahara/mahara/blobs/master/htdocs/lib/cron.php#line488
Additionally, when we try to restart a job which has been running for > 24 hours, we should probably call cron_lock() rather than insert_record()
http://gitorious.org/mahara/mahara/blobs/master/htdocs/lib/cron.php#line498
That said, we're very unlikely to ever hit this given the use of cron.
To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/891170/+subscriptions
References