[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Malcolm Rowe <malcolm-svn-dev_at_farside.org.uk>
Date: 2007-09-05 23:11:18 CEST

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

  • application/pgp-signature attachment: stored
Received on Wed Sep 5 23:08:09 2007

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.