Avoid query for existing artefact record when we know it's new.

Bug #1364687 reported by Nigel Cunningham on 2014-09-03
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Medium
Nigel Cunningham

Bug Description

When importing users from a CSV file, we can avoid the query for an existing artefact record when we know it doesn't exist (because we're being called from create_user). This shaves approximately 40% off the time for creating a user.

Changed in mahara:
assignee: nobody → Nigel Cunningham (nigelc-g)
status: New → In Progress
Changed in mahara:
importance: Undecided → Medium
tags: added: performance

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

commit f0fd10e2c0c0be08c8a8cc2459f2f388d4c548d9
Author: Nigel Cunningham <email address hidden>
Date: Wed Aug 6 15:24:56 2014 +1000

(Bug 1364687) Don't look for an artefact record when we know we're making a new one.

Avoid the query for an existing artefact record when we know it doesn't
exist (because we're being called from create_user). This shaves
approximately 40% off the time for creating a user.

Change-Id: I81697dfd0916e4a2933dfe5d1389bdc649e6e856
Signed-off-by: Nigel Cunningham <email address hidden>

Robert Lyon (robertl-9) on 2015-02-11
Changed in mahara:
milestone: none → 15.04.0
status: In Progress → Fix Committed
Robert Lyon (robertl-9) on 2015-04-17
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers