should cache local string bundles

VERIFIED FIXED in M14

Status

defect
P3
normal
VERIFIED FIXED
20 years ago
11 years ago

People

(Reporter: Bienvenu, Assigned: mscott)

Tracking

({perf})

Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [Perf])

I believe we can get a performance win by caching string bundles. Here is the
call that creates a local string bundle:

E:\raptor\mozilla\mailnews\local\src\nsLocalStringBundle.cpp(48):  res =
sBundleService->CreateBundle(propertyURL, locale, &sBundle);
Whiteboard: [Perf]
Putting on [Perf] radar.
Status: NEW → ASSIGNED
Target Milestone: M11
Target Milestone: M11 → M12
Blocks: 11091
Pls ignore my previous comment.  I am pasting the wrong things into bug reports.
Target Milestone: M12 → M14
Keywords: perf
Bulk add of "perf" to new keyword field.  This will replace the [PERF] we were
using in the Status Summary field.
Adding alecf to the cc list 'cause this was showing up as a hot spot for local
msg display and pop on his quantify work.

I've implemented caching for the local component in my tree. I'll probably land
this tomorrow!
Changing the QA contact to suresh in lieu of lisa.

I checked in a fix for this. The following operations should be faster the next
time Suresh runs his performance data
1) downloading a lot of pop mail. Although there is still a big performance here
we can do. We update the status bar after each message downloaded instead of
after every 10 or so..
2) delete a local message and loading the next message after it should be faster
3) parsing local folders. i.e. delete the summary file for a local mail folder
then select the folder so we re-parse it.


Status: ASSIGNED → RESOLVED
Closed: 20 years ago
QA Contact: lchiang → suresh
Resolution: --- → FIXED
Suresh - check the items that mscott mentions in your next performance tests 
after 1/28.  Compare before and after. Thanks.

Scott - all platforms?
QA Contact: suresh → stephend
I'm sure this got faster with outliner, and since I don't have suresh's data
before me, I'm going to mark this verified.  We have outstanding bugs on message
display speed and folder loading speed, both of which are actively assigned.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.