*IMPORTANT* - bugfix sync_append_commit index breakage

Ken Murchison murch at andrew.cmu.edu
Sat Sep 1 13:08:00 EDT 2007


David Carter wrote:
> On Sat, 1 Sep 2007, Ken Murchison wrote:
> 
>> I applied this patch to CVS, but I'm wondering why the original code 
>> writes directly to the existing cyrus.index instead of a 
>> cyrus.index.NEW, as is done in other places.  I don't know if this was 
>> intentional by the previous authors, or an oversight.  I'm going to 
>> investigate.
> 
> Doesn't cyrus.index.NEW only kick in on expunge?

I believe so.  If we also used it for appends, we wouldn't run into the 
problem that Fastmail is seeing.  But, it may be too much of a 
performance hit.

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


More information about the Cyrus-devel mailing list