Skip to content

Commit

Permalink
MDL-42882 upgrade: improved SQL query
Browse files Browse the repository at this point in the history
Kudos to Tim Hunt who came up with this, on mysql with 4 milion
records (and a fast SSD) the performance difference is:

Original: 36.83 sec
New query: 8.63 sec
  • Loading branch information
danpoltawski committed Feb 3, 2014
1 parent 7ea6574 commit ec62826
Showing 1 changed file with 7 additions and 10 deletions.
17 changes: 7 additions & 10 deletions lib/upgradelib.php
Expand Up @@ -2096,16 +2096,13 @@ function upgrade_fix_missing_root_folders() {
global $DB, $USER;

$transaction = $DB->start_delegated_transaction();
$sql = "SELECT distinct f1.contextid, f1.component, f1.filearea, f1.itemid
FROM {files} f1 left JOIN {files} f2
ON f1.contextid = f2.contextid
AND f1.component = f2.component
AND f1.filearea = f2.filearea
AND f1.itemid = f2.itemid
AND f2.filename = '.'
AND f2.filepath = '/'
WHERE (f1.component <> 'user' or f1.filearea <> 'draft')
and f2.id is null";

$sql = "SELECT contextid, component, filearea, itemid
FROM {files}
WHERE (component <> 'user' OR filearea <> 'draft')
GROUP BY contextid, component, filearea, itemid
HAVING MAX(CASE WHEN filename = '.' AND filepath = '/' THEN 1 ELSE 0 END) = 0";

$rs = $DB->get_recordset_sql($sql);
$defaults = array('filepath' => '/',
'filename' => '.',
Expand Down

0 comments on commit ec62826

Please sign in to comment.