Closed Bug 676247 Opened 9 years ago Closed 8 months ago

Disable places consistency checks as we don't use places

Categories

(MailNews Core :: Database, defect, minor)

x86
All
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: Usul, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: perf)

I just caught the following in the Error console :

[ Places Maintenance ]
> Integrity check
+ The database is sane
> Coherence check
+ The database is coherent

AFAIK we don't use places, so would be nice not to check the db.
Also see bug 672988 on unused places.sqlite file occupying 10MB in the profile.
Summary: Disable palce consistancy checks as we don't use places → Disable places consistency checks as we don't use places
Hi, why do you think TB does not use Places? I thought that too, but then came across bug 677954.
Has something changed?
Component: General → Database
Product: Thunderbird → MailNews Core
QA Contact: general → database
(assuming this perf issue isn't debilitating, or perhaps even noticeable)
Severity: normal → minor
OS: Mac OS X → All
ludo?

(In reply to :aceman from comment #2)
> Hi, why do you think TB does not use Places?
(In reply to Wayne Mery (:wsmwk) from comment #4)
> ludo?
> 
> (In reply to :aceman from comment #2)
> > Hi, why do you think TB does not use Places?

We don't browse and places is to store url and the file never grows - so I don't think we use it.
It looks like we browse, see bug 677954.

The file probably grows only when you reach those 10MB of history entries.
(In reply to :aceman from comment #6)
> It looks like we browse, see bug 677954.

That one looked more like a toolkit bug, which is fixed by now:

(Quoting Mark Banner (:standard8) from bug 677954 comment #7)
> We use some of the interfaces, at most, we could be storing favicon
> information. However, places is disabled by default, so it shouldn't be
> storing history. [...]
> It is slightly surprising that places is allocating memory and doing stuff
> with the db when the places history is disabled.

(Quoting Jonathan Protzenko [:protz] from bug 677954 comment #12)
> We should just revert that change, it's useless now bug 674210 has landed.

If places is disabled in Thunderbird then places.sqlite should be redundant, but given that extensions like Thunderbrowse may keep track of browsing and bookmarks/history, any fix here (and for bug 672988) will have to consider that something may actually want to use it even if it's not Thunderbird itself.

based on previous comments, places does get used. so => wontfix

Blocks: tb-places
Status: NEW → RESOLVED
Closed: 8 months ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.