More Skiplist Stuff!

[David Carter]

On Wed, 2 Jan 2008, Bron Gondwana wrote:

> The renewed one is:
> http://cyrus.brong.fastmail.fm/patches/cyrus-skiplist-transactions-2.3.10.diff
>
> This didn't apply when I was building 2.3.11, and stupid me went "ahh, 
> Ken must have accepted it since it's really quite important to have 
> something like this".  Gosh.  No wonder our mailboxes.db reliability has 
> been dropping off again.

This one still seems to be outstanding. Is it now obsolete given:

> The new one is what found this, and is the _real_ solution to lock
> safety - code that asserts that the locking status of the file is
> what we expect it to be by tracking both the lock status and an
> is_open allowing us to re-read the entire header safely in each
> locking function.
>
> http://cyrus.brong.fastmail.fm/patches/cyrus-skiplist-state-2.3.11.diff
>
> I haven't put a description on this one, but it's reasonably clear
> from the code I hope.

which was merged earlier this week?

-- 
David Carter                             Email: David.Carter at ucs.cam.ac.uk
University Computing Service,            Phone: (01223) 334502
New Museums Site, Pembroke Street,       Fax:   (01223) 334679
Cambridge UK. CB2 3QH.