Re: Testers wanted for FSFS svnadmin recover feature (r23595)

On Wed, Mar 07, 2007 at 11:31:21PM -0500, David Glasser wrote:
> >You're saying the "lock acquired" message is appropriate even for FSFS?
>
> My impression is that printing a "lock acquired" message is
> appropriate for FSFS, but the time it is printed is too early. (The
> current svn_fs_fs__recover implementation wraps most of the
> calculation in a write lock on the repository, but this isn't called
> until *after* svn_svn_recover2 calls the callback which prints the
> message.)
>

This is because the repository layer is reponsible for doing the locking
for a filesystem-layer function. Fixing that would be nice, since it's
only the filesystem that's in a position to know what locking is necessary.

Probably hard to do though.

Regards,
Malcolm