Re: SQLite locking and FSFS rep-sharing

Daniel Shahaf <danielsh_at_elego.de> writes:

> Philip Martin wrote on Mon, Jun 20, 2011 at 12:01:44 +0100:
>
>> We could change the rep-sharing database access to take the FSFS
>> repository lock.

I was mistaken about this bit:

>> Read access to the database already occurs at a point
>> when the lock is held.

Access happens inside svn_fs_fs__set_contents, this normally happens
when the database lock is not held.

> Hmm. Intuitively I wouldn't want sqlite writes to have to take the FSFS
> write lock, since I don't want to block commits on them.
>
> But. SQLite writers block readers, so as long as we write to
> rep-cache.db outside the FSFS write lock (with or without retries) we
> should ensure we don't start readers that hold the FSFS write lock.

-- 
Philip