Bug 333560 - Telepathy contactlist unresponsive when activating grouping of thousand contacts
Summary: Telepathy contactlist unresponsive when activating grouping of thousand contacts
Status: RESOLVED FIXED
Alias: None
Product: telepathy
Classification: Frameworks and Libraries
Component: contactlist (show other bugs)
Version: 0.8.0
Platform: Kubuntu Linux
: NOR minor
Target Milestone: Future
Assignee: Telepathy Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-04-18 01:30 UTC by Fabio Correa
Modified: 2014-04-28 15:36 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 0.8.1


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.