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

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

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Tue, 12 Apr 2011 09:00:27 -0400

> You are looking at changelists as a way to learn how to move operations into
> wc_db properly, but just like that temp table for notifications I don't see
> this as the way to go forward.
>
> I really don't see why users want to add thousands of nodes to changelists
> while we still don't support changelists on directories. And if it is just a
> handful of nodes the old code worked fine.

This was one of the wrestling matches that I had with myself when I started
looking at this very bit of code that Hyrum has changed. As I *understood*
it, we had an internal goal of losing the svn_wc__node_walk_children().
"It's slow." But in some cases -- namely this one -- it just seemed like
doing so would require adding obnoxious or otherwise unpleasant code.

Changelist operations are, I would suspect, pretty rare, so if folks don't
like the approach Hyrum has taken, I would suggest that he just revert the
whole of his effort in this space, delete notes/wc_node_walkers.txt, add a
note to the svn_wc__node_walk_children() docstring encouraging developers to
consider using a more batch-based approach if possible when considering
additional uses of the function, and then move on. If we're going to spin
our wheels somewhere, let's not do it on our arguably lesser-used features,
please.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on 2011-04-12 15:01:06 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.