Issue Details (XML | Word | Printable)

Key: MDL-11180
Type: Task Task
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Martín Langhoff
Reporter: Martin Dougiamas
Votes: 2
Watchers: 7
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

19PERF META BUG: Inspect and merge Catalyst accesslib improvements into HEAD

Created: 10/Sep/07 02:39 PM   Updated: 29/Jan/08 06:02 PM
Return to search
Component/s: Roles
Affects Version/s: 1.9
Fix Version/s: 1.9

Issue Links:
Dependency
 

URL: http://moodle.org/mod/forum/discuss.php?d=79455
Participants: AjnabiZ, Martin Dougiamas and Martín Langhoff
Security Level: None
Resolved date: 29/Jan/08
Affected Branches: MOODLE_19_STABLE
Fixed Branches: MOODLE_19_STABLE

Sub-Tasks  All   Open   
 Sub-Task Progress: 
No sub-tasks match this view.

 Description  « Hide
The performance patches from http://moodle.org/mod/forum/discuss.php?d=79455 need to be examined, tested.

1) Examine new code in various environments and file/fix bugs here as necessary
2) Test common operations such as logging in, listing courses, searching courses, moving around, assigning roles, using modules, switch user, login as on:
      a) small sites
      b) large sites (25,000 students / 10,000 courses / 100,000 enrolments)
     and measure page generation times and database load for current HEAD and new code
3) Along the way keep fixing any divergences in functionality by posting them here.
4) If after all that, if the new code is better overall by, say, 20% or more then let's add it to HEAD ASAP.
5) Continue testing in HEAD.

Make sure you have a look at the testing and dev hints mentioned in the forum thread above.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Martin Dougiamas added a comment - 12/Sep/07 04:10 PM
Can you land these patches in HEAD tomorrow, Martin? (Thursday)

I think it's very clear that the performance improvements are there and it seems to be working . Putting them in HEAD will make further examination and polishing a lot easier for the rest of us.


Martin Dougiamas added a comment - 12/Sep/07 05:19 PM
Actually, Yu iwas a bit worried about MDL-11220 and think a solution might be non-trivial so perhaps if we can solve that first before a merge.

Martín Langhoff added a comment - 12/Sep/07 08:03 PM
Just to assuage any concerns, I am taking MDL-11220 seriously, and I think I have a fix. It's my priority for tomorrow. And – agreed – I wouldn't dream of merging with that bug open.

Martín Langhoff added a comment - 17/Sep/07 10:50 AM - edited
I am going to be trying to finish this off in the next couple of days. Other projects are starting to pull me in all sorts of directions, and hacking on this code needs quite a bit of focus (at least for me!). So – testers/reviewers, hit it hard so I can address all anything you unearth.

Martin Dougiamas added a comment - 18/Sep/07 12:08 PM
Yep, it's looking pretty good. Definitely good enough to merge into CVS. Can you start tackling this please? I'm keen to give 1.9 time to mature with this patch baked in!

AjnabiZ added a comment - 22/Sep/07 12:42 PM
I hope its ready

Martín Langhoff added a comment - 30/Sep/07 04:56 PM
I'm intending to clear these up – including the issues raised in he last couple of days – this week. Got time set aside Monday & Tuesday for it.