Large Mailbox Append Fix

[Bron Gondwana]

On Thu, 20 Sep 2007 10:56:59 +0100 (BST), "David Carter" <dpc22 at cam.ac.uk> said:
> On Thu, 20 Sep 2007, Bron Gondwana wrote:
> 
> > Actually, I'm not 100% sure that it's the only way to do things, but it 
> > was not particularly invasive and a small extra amount of CPU time and 
> > memory copying in exchange for a large saving on replica IO has made our 
> > servers a lot happier about users with big mailboxes being moved!
> 
> Definitely an improvement over the repeatupload stuff. It would make
> sense 
> for upload_messages_from() to also use index_list_work(), although the 
> chance of a monster UPLOAD from a normal append is much lower.
> 
> I would prefer sync_index_list_free() to be called in the parent routine 
> upload_messages_list() rather than as a side effect to index_list_work(), 
> but that is probably just personal style.

Attached is a patch that does both of these :)  It's possible that I haven't
done code-path coverage in my limited testing (it's on our test machine) so
a bit of review is probably in order!

Bron.
-- 
  Bron Gondwana
  brong at fastmail.fm

-------------- next part --------------
A non-text attachment was scrubbed...
Name: cyrus-largemailboxappend-2.3.9.diff
Type: application/octet-stream
Size: 11267 bytes
Desc: not available
Url : http://lists.andrew.cmu.edu/pipermail/cyrus-devel/attachments/20070921/1f326d9f/attachment.obj