Re: svn commit: r1401901 - in /subversion/trunk/subversion: include/private/svn_ra_private.h libsvn_ra/editor.c libsvn_ra/ra_loader.c libsvn_ra/ra_loader.h

[Hyrum Wright <hw...@apache.org>]

On Wed, Oct 24, 2012 at 6:06 PM,  <hw...@apache.org> wrote:
> Author: hwright
> Date: Wed Oct 24 22:06:33 2012
> New Revision: 1401901
>
> URL: http://svn.apache.org/viewvc?rev=1401901&view=rev
> Log:
> First cut at an RA-layer function which returns an Ev2-capable editor for use
> in replay range consumers, such as svnrdump and svnsync.
>
> Note:  This code isn't yet called anywhere or tested, so use at own risk!
>
...

For those wondering, the Grand Plan here goes something like this:
1) Merge these changes to the ev2-export branch (already done in r1401904)
2) On the ev2-export branch, implement the svnrdump editor as an Ev2
provider, using the APIs introduced in this revision.
3) Tie the new RA API into the svn_repos Ev2 replay ranges API for ra_local.
4) Fix bugs.

I'm sure "profit" is in there somewhere, but I'm not sure where it fits. :)

This *should* address issue 4116, but still has a couple of problems,
namely that the RA method will still be using the shims for ra_serf
and ra_svn.  In order to remove that dependency, we'd need a bunch of
protocol work to define a mechanism to punch Ev2 through on both serf
and ra_svn.  While this is probably a sizable task, it can be done in
parallel with (2) above, and will also be useful for things like blame
and update.  Left to my own devices, it will quite some time before
all this gets done, so coconspirators would be appreciated.

-Hyrum