Closed
Bug 243281
Opened 20 years ago
Closed 20 years ago
SortThreads is inefficient
Categories
(MailNews Core :: Backend, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: neil, Assigned: neil)
Details
(Keywords: perf)
Attachments
(2 files)
3.91 KB,
patch
|
Bienvenu
:
review+
mscott
:
superreview+
|
Details | Diff | Splinter Review |
3.56 KB,
patch
|
Details | Diff | Splinter Review |
I've got a news folder with over 250,000 threaded headers. There appear to be three costs associated with opening the folder. The first cost only appears to apply once per session; at least, switching folder to a local folder and back to this news folder does not appear to incur this cost. The second cost appears to apply every time the folder is opened, and can take up to three minutes. The third cost is the sorting of the threads. With the current code, it takes a little over 20 minutes to open the folder for the second time, and almost 20 minutes to resort the folder into a different order (I use size and order received as they are pretty snappy sorts). I can reduce the time taken to sort by order received to half a second, although strangely the time taken to open the folder for subsequent times now varies wildy from one to five minutes.
Assignee | ||
Comment 1•20 years ago
|
||
The precondition never seems to get hit; it appears SortThreads can only be called from Sort when threading is already enabled. As I "collapse" the threads without marking them collapsed I have to set the collapsed bit before re-expanding them unless you know of a better way?
Assignee: sspitzer → neil.parkwaycc.co.uk
Status: NEW → ASSIGNED
Assignee | ||
Updated•20 years ago
|
Attachment #148187 -
Flags: review?(bienvenu)
Comment 2•20 years ago
|
||
can you attach a -uw diff so I can see what's really changed? I'll see if I can hit the code you've removed - you might be right that it's not hit...
Comment 3•20 years ago
|
||
OK, the only place we call sort threads, the view flags do have the threaded bit set, so that change is good.
Assignee | ||
Comment 4•20 years ago
|
||
Comment 5•20 years ago
|
||
Comment on attachment 148187 [details] [diff] [review] Proposed patch thx, that looks good.
Attachment #148187 -
Flags: review?(bienvenu) → review+
Assignee | ||
Updated•20 years ago
|
Attachment #148187 -
Flags: superreview?(mscott)
Updated•20 years ago
|
Attachment #148187 -
Flags: superreview?(mscott) → superreview+
Assignee | ||
Updated•20 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•