Re: svn commit: r16694 - in trunk/subversion: libsvn_client libsvn_ra libsvn_ra_dav libsvn_ra_local libsvn_ra_svn svnserve

On Sun, 16 Oct 2005, Greg Hudson wrote:

> On Sun, 2005-10-16 at 21:50 +0200, Peter N. Lundblad wrote:
> > The problem with just failing in this case is that we spread the handling
> > of this case out to all callers instead of keeping it inside libsvn_ra_svn
> > (which is the only RA layer that doesn't support this function for old
> > "servers"). We have introduced other RA calls that might fail with "not
> > supported" errors, but that's when there is no workaround that's cheap.
>
> I guess I don't consider reopening the connection "cheap".

I didn't say that. My main argument was that if a client reparents, it wil
want to make the connection point to that new FS path, and if the reparent
fails, it will have to reconnect anyway. Why not take that complexity out
of all callers and put it into the only RA layer that needs it?

Regards,
//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Oct 17 07:51:00 2005