[PATCH] Set lock pointer at the start of foreach (Re: 2.3.12 transaction problem within skiplist DB->foreach())

[Rudy Gevaert]

Bron Gondwana wrote:
> On Fri, 29 Aug 2008 09:36:40 -0400, "Wesley Craig" <wes at umich.edu> said:
>> On 29 Aug 2008, at 02:31, Bron Gondwana wrote:
>>> Please find attached a tested patch that fixes the problem and  
>>> refactors
>>> the bloody mess that was locking code into a couple of nice, neat
>>> functions.
>> A quick rewiew looks good to me.  I'll review it closely this  
>> afternoon and deploy it on a test cluster.  Presuming I find no  
>> issues I'll commit it thereafter.  Any chance of submitting it to  
>> bugzilla?
> 
> Sure.  Have done.  Bug 3075.
> 
> Bron.


I just hit the a similar problem (see the thread skiplist errors on 
Cyrus 2.3.12).

Bron, I applied your patch on my test system and the problem doesn't 
happen to me any more.

I was facing the UNLOCKED too when a I deleted a top level mailbox of a 
user.



-- 
-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
Rudy Gevaert          Rudy.Gevaert at UGent.be          tel:+32 9 264 4734
Directie ICT, afd. Infrastructuur ICT Department, Infrastructure office
Groep Systemen                    Systems group
Universiteit Gent                 Ghent University
Krijgslaan 281, gebouw S9, 9000 Gent, Belgie               www.UGent.be
-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --