From: Ramkumar Ramachandra <artagnon@gmail.com>
Date: Thu, 20 Jan 2011 06:28:39 GMT
Subject: Re: [PATCH 4/5] fast-export: Introduce --inline-blobs
Message-ID: <20110120062837.GA27624@kytes>
In-Reply-To: <20110120054813.GB4770@burratino>

Hi Jonathan,

Jonathan Nieder writes:
> Ramkumar Ramachandra wrote:
> 
> > The functionality to persist blobs that are refenced by
> > marks probably shouldn't be in svn-fe at all.
> 
> Do you mean svn-fi?

Yeah. Sorry about the typo. Just to make it clear:

svn-fi should only ever support inlined blobs. For older streams,
there's an extra overhead- it'll have to be chained along with a
helper program that transforms the stream to inline all the blobs.

> > This filter will need to persist every blob for the entire lifetime of
> > the program.
> 
> Depending on the interface, couldn't it be possible to rely on svn for
> the content of blobs that have already been exported?  If so, one
> would only need a place to stash (1) a mapping from mark numbers to
> (svn rev, path) pairs and (2) the full text of blobs that have not
> been exported as part of a rev yet.

Oh yes. We discussed this on IRC :) I'm just afraid that it won't be
fast enough- my idea is to essentially use `svnrdump dump` to replay
the blobs in a certain (path, revision); let me know if you think
there's a quicker way.

-- Ram