Ticket #1485172 (new Patches)

Opened 4 months ago

Last modified 2 months ago

Patch to use imap_root before imap connect (significant speedup)

Reported by: ziba Owned by:
Priority: 5 Milestone: later
Component: IMAP connection Version: svn-trunk
Severity: major Keywords:
Cc:

Description

By communicating the roundcube imap_root preference to iloha imap before connecting to imap, we can take advantage of iloha's ability to skip the namespace discovery phase.

In an environment like ours (80K users, 900K mailboxes), this cuts the time it takes the backend to load a single email message from ~1.7 seconds to ~0.2 seconds (on an unloaded test system).

Please let me know if I can change anything to make this patch more acceptable. Thanks.

Attachments

use_imap_root_before_connect.patch (0.7 kB) - added by ziba 4 months ago.
use_imap_root_before_connect_v2.patch (0.7 kB) - added by maharaja 2 months ago.
updated patch
use_imap_root_before_connect_v3.patch (1.0 kB) - added by ziba 2 months ago.
v2 is missing a semicolon

Change History

Changed 4 months ago by ziba

Changed 2 months ago by maharaja

two things

# if rcmail::get_instance()->config->get('imap_root') gives an empty string, the patch renders roundcube unusable. # normally i am against the usage of globals in such a way. but this is better than nothing :)

Changed 2 months ago by maharaja

updated patch

Changed 2 months ago by ziba

v2 is missing a semicolon

Note: See TracTickets for help on using tickets.