Closed
Bug 554015
Opened 14 years ago
Closed 13 years ago
reorder fulltext columns so body is the 0th column for disk-space savings
Categories
(MailNews Core :: Database, defect)
MailNews Core
Database
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 10.0
People
(Reporter: asuth, Assigned: protz)
Details
(Keywords: perf, Whiteboard: [gloda key])
Attachments
(1 file, 1 obsolete file)
6.03 KB,
patch
|
asuth
:
review+
|
Details | Diff | Splinter Review |
Per http://www.sqlite.org/fts3.html, the doclist encoding for terms encountered in the 0th column is specialized. This would save us 2 bytes per distinct term per document in the body. It would also cost us 2 bytes per term per document in the subject, but since the subject should generally be much shorter than the body, the choice is clear.
Assignee | ||
Comment 1•13 years ago
|
||
This seems to make test_msg_search.js fail (the non-failing tests might by commented out). Any idea as to why this may be failing will be highly appreciated :-).
Assignee: bugmail → jonathan.protzenko
Attachment #551971 -
Flags: feedback?(bugmail)
Assignee | ||
Comment 2•13 years ago
|
||
Ok, found what I missed. I fixed the comment if we ever want to swap the two again ;-)
Attachment #551971 -
Attachment is obsolete: true
Attachment #552003 -
Flags: review?(bugmail)
Attachment #551971 -
Flags: feedback?(bugmail)
Reporter | ||
Comment 3•13 years ago
|
||
Comment on attachment 552003 [details] [diff] [review] Patch v2 Ah yeah, the ranker function was a more recent addition. Looks great, thanks for taking care of this!
Attachment #552003 -
Flags: review?(bugmail) → review+
Reporter | ||
Comment 4•13 years ago
|
||
I suspect you are already aware, but just to make sure, we cannot land this unless we also land a patch that bumps the schema version, so do keep it in your queue. I think it's okay if we bump the schema multiple times on comm-central as long as it doesn't get shifted to comm-aurora in between, so let's wait to do that until just after we make the next cut from comm-central to comm-aurora which should be happening soon.
Assignee | ||
Comment 5•13 years ago
|
||
http://hg.mozilla.org/comm-central/rev/5273b72d2e58
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 10.0
You need to log in
before you can comment on or make changes to this bug.
Description
•