remove empty threads from db when last msg header is deleted to improve performance

RESOLVED FIXED in Future

Status

MailNews Core
Database
P2
normal
RESOLVED FIXED
18 years ago
10 years ago

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Tracking

(Blocks: 1 bug, {perf})

Trunk
Future

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [nsbeta1+ 2/13])

(Assignee)

Description

18 years ago
When we delete the last header from a thread, we should delete the thread table
from the underlying MDB database.
(Assignee)

Comment 1

18 years ago
this will improve performance by making databases smaller, and reducing the
memory footprint of an open database.
Status: NEW → ASSIGNED
Keywords: perf
(Assignee)

Updated

18 years ago
Keywords: mail1
(Assignee)

Comment 2

18 years ago
changing priorities
Priority: P3 → P2

Comment 3

18 years ago
masrking nsbeta1+ and moving to mozilla0.8
Keywords: nsbeta1
Whiteboard: [nsbeta1+]
Target Milestone: --- → mozilla0.8

Comment 4

18 years ago
moving to mozilla0.9
Target Milestone: mozilla0.8 → mozilla0.9

Comment 5

17 years ago
marking nsbeta1- and moving to future milestone.
Keywords: nsbeta1 → nsbeta1-
Whiteboard: [nsbeta1+] → [nsbeta1+ 2/13]
Target Milestone: mozilla0.9 → Future

Updated

14 years ago
Blocks: 236849
Product: MailNews → Core

Comment 6

11 years ago
this still true?
Status: ASSIGNED → NEW
OS: Windows NT → All
QA Contact: esther → database
Hardware: PC → All
Summary: remove empty threads from db when last msg header is deleted → remove empty threads from db when last msg header is deleted to improve performance
(Assignee)

Comment 7

11 years ago
I don't know if this is still true. My recollection is that Mork is supposed to do this for us (delete empty tables with no references to them) and it may do so. 
This may not be true, based on my 16913 tests. I'll have to look into it more....
I can't quite tell if mork does this or not (there is a lot of code marked "to be fixed later" or similar in mork).

But you actually fixed this back in 2002 as a result of bug 140613...
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.