Large cache miss ratio due to duplicate UIDs

[patrys]

Roundcube uses a pair of (uid, cache_key) as the uniqueness enforcer in the messages table. Some systems (exim + courier-imap here) do not guarantee global UID uniqueness, UIDs are only unique per mailbox.

Since RC checks for the user_id match anyway, I was able to temporarily fix it (large numbers of DB Error: Already exists) by adding user_id as the third column to the UNIQUE key.

[kajtzu@…]

This seems to happen with cyrus-imapd as well and svn-trunk.

[moc]

I have the same problem with Binc IMAP. A possible fix is to add the user_id field to the UNIQUE key for the messages table.

Here is the patch for PostgreSQL: ALTER TABLE "messages" ADD UNIQUE (user_id, cache_key, uid);

[thomasb]

Fixed in Trunk (r509)