Closed Bug 66576 Opened 24 years ago Closed 23 years ago

use batching to speed up multiple message delete or move

Categories

(SeaMonkey :: MailNews: Message Display, defect, P2)

defect

Tracking

(Not tracked)

VERIFIED DUPLICATE of bug 80897
Future

People

(Reporter: sspitzer, Assigned: sspitzer)

References

Details

(Keywords: perf, Whiteboard: [nsbeta1+ 2/21])

Attachments

(2 files)

patch coming soon, for review and test.
Reminder to myself, report the performance metrics for this when I test next against our pre-Pierre results.
Keywords: perf
QA Contact: esther → stephend
accepting, here comes the patch.
Status: NEW → ASSIGNED
Attached patch patch (deleted) — Splinter Review
in my local tree, I'm trying out a slight variation of that patch, one where I call endBatch() after we select the next message. I'll attach it. either one, I think I've found a small bug with tree batching. I'll log a new bug on that, and try to find some time to discuss it with hyatt.
Priority: -- → P2
Target Milestone: --- → mozilla0.8
marking 0.8, p2.
Keywords: nsbeta1
Whiteboard: [nsbeta1+]
Perhaps it's about time this patch gets a r=/sr=?
This can't be checked in until 66583 is fixed, so there's not much point in reviewing this.
moving to mozilla0.9
Target Milestone: mozilla0.8 → mozilla0.9
Seth does this bug go away with the thread pane rewrite?
marking nsbeta1- and moving to future milestone. I think this might get taken care of with the rewrite. If not and there's something we can do here to speed us up then we should still consider doing it.
Keywords: nsbeta1nsbeta1-
Whiteboard: [nsbeta1+] → [nsbeta1+ 2/21]
Target Milestone: mozilla0.9 → Future
*** Bug 83656 has been marked as a duplicate of this bug. ***
updating summary.
Summary: use tree batching apis to speed up multiple message delete or move → use batching to speed up multiple message delete or move
note, I'm sure those patches are way out of date. we'd still like to do this. even with outliner, multiple deletes are slow.
*** Bug 115220 has been marked as a duplicate of this bug. ***
*** Bug 73758 has been marked as a duplicate of this bug. ***
Is this a duplicate of Bug 80897 ?
It looks like it. *** This bug has been marked as a duplicate of 80897 ***
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
verified dup.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: