Refactor set_access process for efficiency

Bug #1952961 reported by Doris Tam
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Status tracked in 22.04
21.10
High
Unassigned
22.04
High
Doris Tam

Bug Description

The set_access function takes a very long time. Need to reduce Big-O complexity.

This issue has surfaced from trying to upgrade a site with many collections. We're getting stuck at the db upgrade step 'Add "progress" page type' when needing to process 4000 collections and setting pages access for each the pages.

Doris Tam (doristam)
Changed in mahara:
assignee: nobody → Doris Tam (doristam)
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/12282
Committed: https://git.mahara.org/mahara/mahara/commit/9c9388315dbe07806ebf5faa9284512fb18c5fba
Submitter: Robert Lyon (<email address hidden>)
Branch: main

commit 9c9388315dbe07806ebf5faa9284512fb18c5fba
Author: Doris Tam <email address hidden>
Date: Wed Dec 1 17:28:05 2021 +1300

Bug 1952961: Reduce complexity of set_access

Change-Id: Ifd728955324ad6a39dd1123e699444e739f4d5b4

Robert Lyon (robertl-9)
Changed in mahara:
milestone: none → 22.04.0
importance: Undecided → High
status: New → Fix Committed
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "21.10_DEV" branch: https://reviews.mahara.org/12287

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

Reviewed: https://reviews.mahara.org/12287
Committed: https://git.mahara.org/mahara/mahara/commit/ad32956194e7246fe90af91bf6f7e4a2a14f867c
Submitter: Robert Lyon (<email address hidden>)
Branch: 21.10_DEV

commit ad32956194e7246fe90af91bf6f7e4a2a14f867c
Author: Doris Tam <email address hidden>
Date: Wed Dec 1 17:28:05 2021 +1300

Bug 1952961: Reduce complexity of set_access

Change-Id: Ifd728955324ad6a39dd1123e699444e739f4d5b4

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers