[Tickets #14039] Kolab: Initial folder sync is done twice

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: https://bugs.horde.org/ticket/14039
------------------------------------------------------------------------------
  Ticket             | 14039
  Created By         | Thomas Jarosch <thomas.jarosch at intra2net.com>
  Summary            | Kolab: Initial folder sync is done twice
  Queue              | Horde Framework Packages
  Type               | Bug
  State              | Unconfirmed
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


Thomas Jarosch <thomas.jarosch at intra2net.com> (2015-07-01 14:00) wrote:

Hi Michael,

using the new Token sync based strategy, I noticed the initial sync is  
done twice.
Looks like we don't store the initial token / stamp before doing  
another full, initial sync.

---------------------------------
[root at horde-git tmp]# egrep "Full|token" horde.log
2015-07-02T15:50:42+02:00 DEBUG: HORDE [turba] [KOLAB_STORAGE]  
Fetching stamp without token. [pid 25205 on line 248 of  
"/datastore/DEVEL/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Data/Cached.php"]
2015-07-02T15:50:42+02:00 DEBUG: HORDE [turba] [KOLAB_STORAGE] Full  
folder sync details: user: mueller, folder: INBOX/Contacts, uids:   
[pid 25205 on line 248 of  
"/datastore/DEVEL/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Data/Cached.php"]
2015-07-02T15:50:42+02:00 DEBUG: HORDE [turba] [KOLAB_STORAGE] Full  
Horde_History sync for user: mueller, folder: INBOX/Contacts,  
is_reset: 0, prefix: turba:sZ_eVtpyGKcNvfK4lzOlMs6: [pid 25205 on line  
73 of  
"/datastore/DEVEL/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Data/Query/History/Cache.php"]
2015-07-02T15:50:42+02:00 DEBUG: HORDE [turba] [KOLAB_STORAGE]  
Fetching stamp without token. [pid 25205 on line 248 of  
"/datastore/DEVEL/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Data/Cached.php"]
2015-07-02T15:50:42+02:00 DEBUG: HORDE [turba] [KOLAB_STORAGE] Full  
folder sync details: user: mueller, folder: INBOX/Contacts, uids:   
[pid 25205 on line 248 of  
"/datastore/DEVEL/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Data/Cached.php"]
---------------------------------


The full sync request is probably in the code for ages but was never  
noticed before:
The time based sync triggers only every XX seconds and the session  
based sync triggers one per login only.

Personally I don't mind triggering multiple sync requests from the  
specific Kolab drivers right now, we should just make use of the token  
/ stamp if we already have it. Otherwise huge folders will be twice as  
slow to sync as they need to be :)

Attached is a debug log from a newly created user "mueller" +  
completely cleared Horde_Cache.

Cheers,
Thomas




Thomas Jarosch <thomas.jarosch at intra2net.com> (2015-07-01 14:00)  
uploaded: horde.log

https://bugs.horde.org/h/services/download/?app=whups&actionID=download_file&file=horde.log&ticket=14039&fn=%2Fhorde.log