Large Mailbox Append Fix

Bron Gondwana brong at fastmail.fm
Sat Sep 22 22:23:02 EDT 2007


On Sat, 22 Sep 2007 10:44:55 +0100 (BST), "David Carter" <dpc22 at cam.ac.uk> said:
> On Sat, 22 Sep 2007, Bron Gondwana wrote:
> 
> > We still need to sync the final last_uid, which may be greater than
> > the UID of the maximum message.
> 
> Oops, yes. Replacing last_uid for the last batch should be good enough.
> 
> > (though I think there's a separate check later that does that if they
> > still don't match... I really should test-case this!)
> 
> I don't think so. There is a separate test for UIDLAST if 
> do_mailbox_work() finds that it has no messages to UPLOAD.

Yeah, you're right.  I've got a simpler rewrite of the patch here that
does update_uidlast(mailbox) at the end if there's nothing in the
index_list.

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: 11549 bytes
Desc: not available
Url : http://lists.andrew.cmu.edu/pipermail/cyrus-devel/attachments/20070923/947d1392/attachment.obj 


More information about the Cyrus-devel mailing list