Re: [PATCH] fast-import: Increase the default pack depth to 50
On Thu, Jun 08, 2017 at 04:11:52PM +0900, Mike Hommey wrote:
Show 13 quoted lines
> > It probably doesn't matter that much, though, as you'd really want to
> > `repack -f` afterwards if you care about getting good deltas. And one
> > base object every 50 versions is probably fine for keeping the initial
> > pack manageable.
>
> It actually is possible to have non-linear delta chains with
> fast-import, because the cat-blob command resets the delta base when
> storing a blob. See
> https://github.com/git/git/blob/v2.13.1/fast-import.c#L2984-L2987
>
> As a side effect of that, git-cinnabar[1] ends up with decent-ish delta
> chains out of the box, without having to go with repack -f, when the
> mercurial server gives out changegroup version 2.
Interesting. Today I learned.
I agree that bumping the "10" to "50" is doubly important then.
-Peff
Jeff King <peff@peff.net>