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

Re: RFE: svn_client_ls() should return locking info

From: SteveKing <steveking_at_gmx.ch>
Date: 2005-05-01 18:07:52 CEST

Ben Collins-Sussman wrote:

> Aha, good point. So then the commandline client and TSVN would be equal
> in this feature -- both browsing remote locks via 'status -u'. Perhaps
> in 1.3, we can improve svn_client_ls() as you suggested, and then both
> TSVN's repository browser and 'svn ls -v' would show remote locks.
>
> Anyway, it would be nice to know exactly how big the performance hit
> is... it would be nice if you tried to show locks in ther repos browser,
> just to see.

I already did this morning (hey, it's sunday and nice weather on the
balcony - real nice if you have WLAN ;) ).
On the first try, the performance hit was immense. But then I called
svn_ra_get_locks() only once on repo browser startup (together with
svn_ra_get_repos_root(), because the paths returned by
svn_ra_get_locks() are relative to the repository root). That way, the
performance hit isn't that bad, because all locks are fetched
recursively in one call and I can cache them in a map until I need them.
The only problem now is that the repo browser doesn't show up
immediately anymore without any GUI information showing. But that's of
course a TSVN issue and not a Subversion one. I hope I can implement
something there so the users can at least see that something is going on
and they just have to wait a few seconds.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun May 1 18:08:51 2005

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