*IMPORTANT* - bugfix sync_append_commit index breakage

Ken Murchison murch at andrew.cmu.edu
Fri Aug 31 12:56:35 EDT 2007

David Carter wrote:
> On Sat, 1 Sep 2007, Bron Gondwana wrote:
>> On a more structural note - it would be great to re-factor the lower
>> level APIs (mailbox.c, mboxlist.c, etc) to provide everything that
>> sync_server needs manipulate their data structures and remove all the
>> low level fiddling copy-and-pasted stuff from the sync layer.  This
>> comes back a bit to David's comments today about the difficulty of
>> having patches flying around - I suspect he copied and pasted so he
>> wasn't throwing tentacles deep through the main codebase - but the
>> end result is more brittle software and poor abstraction.
> Pretty much. I wasn't expecting the replication code to ever get merged 
> when I first wrote it. All of the really awful stuff should be in 
> sync_commit.c. Those are the routines which create messages with 
> predetermined UIDs and mailboxes with predetermined UniqueIDs.

I agree, that the code could use a little housecleaning at this point, 
but I'd like to get all of the wrinkles ironed out first, before we 
start making wholesale changes.

Kenneth Murchison
Systems Programmer
Project Cyrus Developer/Maintainer
Carnegie Mellon University

More information about the Cyrus-devel mailing list