Cyrus 2.3.12 RC1

[Bron Gondwana]

On Wed, 9 Apr 2008 13:51:10 +0100 (BST), "David Carter" <dpc22 at cam.ac.uk> said:
> On Wed, 9 Apr 2008, Ken Murchison wrote:
> 
> > Is there a bug somewhere in the rest of the code that causes this 
> > problem?
> 
> It is mostly a sanity check to stop people from accidentally writing code 
> which contains nested transactions. That is actually fairly easy to do 
> given that each mboxlist_findall() quietly starts a transaction.

Or to stop bugs from triggering nested transactions.

> We use this patch as well. The only curiosity that I haven't tracked down 
> is the following from cyr_expire:
> 
> Apr  9 03:43:52 cyrus-35 cyr_expire[15410]: skiplist: unlock while not
> locked
> 
> (I haven't touched cyr_expire in 2.3.11).

Hmm - you're right.  We're getting those too.  I haven't been watching
those log entries closely.  I'll check that out.

(along with finding out why seen DBs get started twice)

Bron.
-- 
  Bron Gondwana
  brong at fastmail.fm