Large Mailbox Append Fix

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

On Sat, 22 Sep 2007 10:44:55 +0100 (BST), "David Carter" <dpc22 at> 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

  Bron Gondwana
  brong at

-------------- 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 : 

More information about the Cyrus-devel mailing list