Bug 682706 - WebDAV addressbook lookup very slow
WebDAV addressbook lookup very slow
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Contacts
3.2.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: evolution-addressbook-maintainers
Evolution QA team
:
: 683649 (view as bug list)
Depends on:
Blocks:
  Show dependency tree
 
Reported: 2012-08-26 00:25 UTC by John Pye
Modified: 2012-09-11 14:01 UTC (History)
3 users (show)

See Also:
GNOME target: ---
GNOME version: ---


Attachments
eds patch (5.17 KB, patch)
2012-08-28 12:25 UTC, Milan Crha
committed Details | Diff | Review

Description John Pye 2012-08-26 00:25:11 UTC
I am using a CardDAV remote addressbook hosted by fruux.com, but the caching of the addressbook appears not to be working, or is not completely implemented. The result is that when using the addressbook, time-lags of 20 seconds or more occr very frequently. 

I first reported this bug at Launchpad, see here:
https://bugs.launchpad.net/ubuntu/+source/evolution/+bug/1032892

Is something missing in the caching of CardDAV addressbooks? Surely the cache should be hit in the first instance, and the updating/querying should be done in a separate thread?
Comment 1 André Klapper 2012-08-27 10:37:50 UTC
Retesting with 3.4 is welcome as 3.2 is rather old nowadays.
Comment 2 Milan Crha 2012-08-27 16:28:08 UTC
I can confirm this bug report, based on the code reading. If the backend is in online mode, then it checks for changes first, and only after that returns contacts based on the downloaded changes, if any. It can/should do as you suggested.
Comment 3 Milan Crha 2012-08-28 12:25:46 UTC
Created attachment 222628 [details] [review]
eds patch

for evolution-data-server;

This makes the lookup responses quicker, it returns cached contacts first, then tries to "validate" output against the server (synchronizes local cache with the server).
Comment 4 Milan Crha 2012-08-28 12:27:01 UTC
Created commit eeb2311 in eds master (3.5.91+)
Comment 5 Milan Crha 2012-09-11 14:01:21 UTC
*** Bug 683649 has been marked as a duplicate of this bug. ***

Note You need to log in before you can comment on or make changes to this bug.