Re: [PATCH] Skip-deltas, for review

William Uther wrote:

>Note that I've moved the "prevnode = node" inside the outer loop. This
>means you re-deltify every 2^Nth revision rather than every 2^(N+1)-1th
>revision. The difference is small. I was just wondering if it was
>intentional.
>
>2^N: 1 2 4 8 16 32
>2^(N+1)-1: 1 3 7 15 31 63
>
>

The second series makes more sense -- you don't want to redeltify
successive revisions (1 and 2 in the first series).

However, I'd like to point out that there's a simpler way than skip
deltas to speed up access to old versions, and keep commit times
reasonably fast. Simply store every N'th revision of a file as a
self-referent delta -- actually, a vdelta-compressed fulltext.

The delta combiner will already stop the walk if it finds a window that
doesn't refer to source data.

-- 
Brane Čibej   <brane_at_xbc.nu>   http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org