Re: Improve svnsync performance over ra_serf.

On Nov 4, 2007 2:25 PM, David Glasser <glasser@davidglasser.net> wrote:
> On 11/4/07, Lieven Govaerts <svnlgo@mobsol.be> wrote:
> > > I'll point out that since the fallback here is just equivalent to "run
> > > svn_ra_replay" a bunch of times, and that's what ra_local and ra_neon
> > > need to do also, it might be a good idea to stick a project-private
> > > function in the ra-loader which simulates svn_ra_replay_range in terms
> > > of svn_ra_replay.
> >
> > I've implemented a client-side fallback in r27584, -5. While this
> > implementation works, it's not ideal. Other clients directly using the
> > ra api's will have to implement a similar wrapper or not use
> > replay_range. The alternative is to do this in the ra layer, but that
> > would require a place to share ra code (libsvn_ra_util?), which I didn't
> > want to add right now.
>
> Hmm, can the client actually differentiate between "replay-range
> unimplemented" and "replay unimplemented"? I'm not sure how they
> would without making up a new error code. I'd lean towards just
> adding a compat function in ra-loader.c...

Given that you've asked for review, I'll point out that you haven't
answered this question...

--dave

-- 
David Glasser | glasser_at_davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org