Performance improvements for cron job

Bug #1427901 reported by Aaron Wells
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Medium
Aaron Wells

Bug Description

For a client, I recently implemented some performance improvements to the Mahara cron job. I've opened this bug to upstream them.

The specific improvements are:

1. Make it so that the cron doesn't run auth_setup()

2. Make it so that the cron doesn't run while the site is in the middle of an upgrade (this can cause a pile-up of cron tasks waiting for a DB lock to be released)

3. Removing the obsolete "forcelocalupgrade" option.

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/4350

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/4350
Committed: http://gitorious.org/mahara/mahara/commit/b4c1755f02d1be190e4c2b8ab8f83293d0c84b7f
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit b4c1755f02d1be190e4c2b8ab8f83293d0c84b7f
Author: Aaron Wells <email address hidden>
Date: Mon Jan 26 17:38:09 2015 +1300

Bug 1427901: Performance improvements for cron

- Cron doesn't need to run auth_setup()
- Don't run cron when site is closed for upgrade
- Get rid of forcelocalupgrade() option because it's no longer needed

Change-Id: I1718b13337c50fadc0573d04f5b3d6b20bc842c2

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "15.04_STABLE" branch: https://reviews.mahara.org/4530

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/4530
Committed: http://gitorious.org/mahara/mahara/commit/0adc97975216929b3bb4a7d681798bf6e76a6ce1
Submitter: Robert Lyon (<email address hidden>)
Branch: 15.04_STABLE

commit 0adc97975216929b3bb4a7d681798bf6e76a6ce1
Author: Aaron Wells <email address hidden>
Date: Mon Jan 26 17:38:09 2015 +1300

Bug 1427901: Performance improvements for cron

- Cron doesn't need to run auth_setup()
- Don't run cron when site is closed for upgrade
- Get rid of forcelocalupgrade() option because it's no longer needed

Change-Id: I1718b13337c50fadc0573d04f5b3d6b20bc842c2

Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.