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

Re: svn commit: r1088958 - /subversion/trunk/subversion/libsvn_wc/wc_db.c

From: Greg Stein <gstein_at_gmail.com>
Date: Mon, 2 May 2011 02:17:59 -0400

On Tue, Apr 5, 2011 at 06:25, <rhuijben_at_apache.org> wrote:
> Author: rhuijben
> Date: Tue Apr  5 10:25:54 2011
> New Revision: 1088958
>
> URL: http://svn.apache.org/viewvc?rev=1088958&view=rev
> Log:
> Make svn_wc__db_op_copy() perform its work inside a sqlite lock; this should
> speed up the performance of 'svn cp' more then just a bit.

Why is the second lookup done within the lock? It seems that both
source and destination lookup should (or can) be done outside of the
lock.

I can see doing the second with_txn() from within the other. But it
seems that the lookup should be done "outside".

>...

Thoughts?

Cheers,
-g
Received on 2011-05-02 08:18:37 CEST

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.