[DOGFOOD] news folders / unread set killing startup performance.

VERIFIED FIXED in M11

Status

defect
P1
normal
VERIFIED FIXED
20 years ago
2 years ago

People

(Reporter: sspitzer, Assigned: Bienvenu)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [Perf])

>     I was trying to track down some memory leaks and I noticed that
> when we load newsrc files, we open all the databases in the newsrc
> file in order to set the unreadSetStr. This defeats the purpose of the
> folder cache (panacea.dat), which is to avoid opening any databases
> until the user clicks on a folder or newsgroup. Would it be possible
> to delay setting the unreadSetStr for the database until the newsgroup
> is opened? This would help startup performance for people with lots of
> newsgroups.
>
> This could work by caching the unreadSetStr in the nsMsgNewsFolder,
> and then when the DB is opened for other reasons, setting the unread
> set there.
Status: NEW → ASSIGNED
Hardware: PC → All
Target Milestone: M9
marking m9.  accepting

Updated

20 years ago
Whiteboard: [Perf]

Comment 2

20 years ago
Putting on [Perf] radar.
Target Milestone: M9 → M10
slipping to m10.

Updated

20 years ago
Target Milestone: M10 → M11

Comment 4

20 years ago
Triage to M11

Comment 5

20 years ago
I'm going to raise this to P1.  This is really hurting us.  I ran Quantify and
this bug added 5 seconds to my debug build and I only have a few newsgroups that
have db's.  I imagine that if all of my newsgroups had db's that this would be a
killer.  Anyway, right now we supposedly take twice as long as the browser to
startup and I imagine that this is one of the things that is doing it to us.
Assignee

Updated

20 years ago
Assignee: sspitzer → bienvenu
Status: ASSIGNED → NEW
Assignee

Comment 6

20 years ago
Seth, I'll take a crack at this.
Assignee

Comment 7

20 years ago
I've got a fix coded, and I'm just checking it out.
Assignee

Comment 8

20 years ago
OK, my fix for this works. I'll check it in when the tree opens. I basically did
what I suggested above, which is to cache the unreadset in the news folder, and
when we open the db, assign it to the opened db.
Status: NEW → ASSIGNED
Assignee

Updated

20 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 20 years ago
Resolution: --- → FIXED

Comment 9

20 years ago
QA has no resources to verify this bug. Please verify it or advise us  how to
verify it. Thanks.

Updated

20 years ago
QA Contact: lchiang → suresh

Comment 10

20 years ago
Fenella, Suresh will handle this.

Suresh, can you work with one the guys to verify this?  Thanks.
QA Contact: suresh → stephend
I'd like to get this radar, either seth or david, could you verify that this is
fixed? Thanks!
verified.
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.