Re: svn commit: r26471 - in trunk: . notes subversion/include subversion/libsvn_client subversion/libsvn_repos subversion/libsvn_wc subversion/tests/cmdline

[Malcolm Rowe <ma...@farside.org.uk>]

On Wed, Sep 05, 2007 at 01:49:05PM -0700, sussman@tigris.org wrote:
> Merge copy-on-updates branch to trunk.
> 

Nice.  Does it work? (the only reason I ask is that you only added a
single XFail test, so I'm not sure what the status of the branch is
currently).

One other comment: it'd be nice if we were able to make libsvn_client's
file_fetcher slightly smarter, so that it's responsible for opening the
second RA session on-demand.  Otherwise, we'll always open an extra
connection to the server every update, even if we don't need it for
anything.  This seems like a waste of server resources.

(It's not clear what the scope of POOL in the svn_wc_get_file_t callback
is, but it's reasonable to assume we can't use it to allocate the
connection: we'd need to stash an appropriate pool somewhere, I guess.)

Regards,
Malcolm