Bug 333560

Summary:	Telepathy contactlist unresponsive when activating grouping of thousand contacts
Product:	[Unmaintained] telepathy	Reporter:	Fabio Correa <facorread>
Component:	contactlist	Assignee:	Telepathy Bugs <kde-telepathy-bugs>
Status:	RESOLVED FIXED
Severity:	minor	CC:	kde, mklapetek
Priority:	NOR
Version:	0.8.0
Target Milestone:	Future
Platform:	Kubuntu
OS:	Linux
Latest Commit:		Version Fixed In:	0.8.1
Sentry Crash Report:

Description Fabio Correa 2014-04-18 01:30:19 UTC

I have near 3000 contacts in my accounts. I keep grouping disabled at the contact list because if I ever activate it, the application becomes completely unresponsive while checking for each contact's groups. In my case I have to wait for up to 10 minutes until I am able to use the application again.

Reproducible: Always

Steps to Reproduce:
1. Enable your Facebook, Google, Skype, and other accounts that add up to more than 1000 contacts.
2. Enable groups: Click the configuration button to the right, click menu option "Contact Grouping" and "Show contacts by groups"
3. Disable groups.
Actual Results:  
When groups are disabled, the UI is pretty fast.
When groups are activated, the UI is unresponsive for several minutes.

Expected Results:  
The UI to be unresponsive for some seconds at most. The contactlist is supposed to be querying the in-memory the contacts to see their tags/groups. This should be pretty fast.

Workaround: Not using the grouping feature.

This is just a minor annoyance; everything else is fantastic about KDE Telepathy. Keep up the work.

Comment 1 David Edmundson 2014-04-18 02:04:02 UTC

0.8.1 (coming this week?) will make this quite a bit faster as it no longer updates every item's groups in turn after the initial load.

Could you retest after that and close if the performance is good enough.

Comment 2 David Edmundson 2014-04-20 22:44:45 UTC

You were right, there is another silly method in the model, which is making things somewhat slow. It's been there since ~0.4 :S
Fix here https://git.reviewboard.kde.org/r/117668/

I think it might be too large to merge before 0.9. 
Martin, what do you think?

Comment 3 Martin Klapetek 2014-04-22 08:33:15 UTC

Doesn't seem /too/ large for me, but if you have concerns, then defer to 0.9.

Comment 4 David Edmundson 2014-04-25 09:47:22 UTC

Merged into 0.8.1. I want to say this is fixed.

 In either case could you please report back with your findings once you get the release.

Comment 5 Fabio Correa 2014-04-28 15:36:06 UTC

Thanks for your work. Tested your git commit 593a1a48 and grouping is way more responsive now.