Re: Should mod_dav_svn/ra_dav support svndiff1? (was: Re: Should 'svnadmin dump --deltas' write svndiff1 diffs?)

On Fri, Feb 10, 2006 at 09:47:21AM -0500, Daniel Berlin wrote:
> > Though that does make me wonder: did we gain any benefit from making
> > ra_dav support svndiff1 diffs in the transport, when the whole stream
> > is almost certainly going to be compressed? Or did we actually make
> > things worse?
>
> It depends on a lot of factors :).
>

Sure. We really need to measure it :)

> IIRC, isn't the data sent base64 encoded by us beforehand anyway?
>

Er, not entirely sure. I think that either we or Neon base64 the
binary blob before transmission, yes. So that would mean that we need
to determine whether

(((svndiff ins. || svndiff data) => base64) || plaintext) => zlib

was significantly worse or better than

(((svndiff ins. || svndiff data => zlib) => base64) || plaintext) => zlib

... (or something like that, anyway)

_If_ (big if) the tests I performed with 'dump --deltas' are in any
way representative, we'll actually send more data in the second case,
and burn more CPU time, too.

(I think that we can agree that it's always better to use svndiff1 if
the HTTP stream itself isn't compressed).

> Or am i just off my rocker?
>

Isn't that orthogonal to the above question? :)

Regards,
Malcolm

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Feb 10 16:32:35 2006