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