Open Bug 690707 Opened 10 years ago Updated 6 years ago

TB7 sorts messages by From too slowly, if "show only display names" is enabled and many contacts are defined in addressbook

Categories

(Thunderbird :: Folder and Message Lists, defect)

7 Branch
x86_64
Windows 7
defect
Not set
normal

Tracking

(Not tracked)

People

(Reporter: groucho43, Unassigned)

References

Details

(Keywords: perf, regression, Whiteboard: [workaround: disable "show only display names"])

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:6.0.2) Gecko/20100101 Firefox/6.0.2
Build ID: 20110902133214

Steps to reproduce:

Installed TB7b3.


Actual results:

Even with the latest Beta of TB7, the sorting of messages in a given folder by the sender's name takes an inordinate amount of time (from 5 to 30 seconds, depending on the number of messages in the folder). In TB3 it takes less than a second. This is a serious problem which will prevent users from upgrading to TB7 or will make them revert to TB3.


Expected results:

TB7 should sort messages by the From header as fast as TB3.
Keywords: perf
Component: General → Folder and Message Lists
QA Contact: general → folders-message-lists
P.S. If the sorting of messages in the From column were done by the To: header, as in TB3, rather than by the Display Name, it would probably be just as fast.
I recall Bienvenu mentioning this in a patch, so CCing him here. For what it's worth, I don't see any speed issues with sorting.
this is caused by Bug 672022 - it should only be the first sort that's slow, and that slowness is directly proportional to the total number of addresses in your address books. If you turn off "show only display names", sorting should revert to its earlier performance (tools, options, advanced, reading and display). And if you make changes to your address book, that will make us have to re-recache the display name and make sorting slow again the next time.

Fix for this will have to wait for a new AB backend that gives us a real index so we can look up display names via an index. Mike, this is the particular problem I was talking about where we need an index of the lower case version of all the e-mail addresses in a card.
Many thnaks, David.
Er... Thanks :-)
Blocks: 672022
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: regression
> Mike, this is the particular problem I was talking about where we need an index of 
> the lower case version of all the e-mail addresses in a card.

*nods*, noted.
To bug opener:
Same as or similar to Bug 672022?
Do you agree with duping of your report to that bug?
See also Bug 695327 in which the Bug 672022 is pointed/mentioned.
Duplicate of this bug: 695327
Summary: TB7 sorts messages by From too slowly → TB7 sorts messages by From too slowly, if "show only display names" is enabled and many contacts are defined in addressbook
Whiteboard: [workaround: disable "show only display names"]
Duplicate of this bug: 739363
You need to log in before you can comment on or make changes to this bug.