Re: [PATCH] fast-import: Increase the default pack depth to 50

2017-06-08 Thread Jeff King
On Thu, Jun 08, 2017 at 04:11:52PM +0900, Mike Hommey wrote: > > 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

Re: [PATCH] fast-import: Increase the default pack depth to 50

2017-06-08 Thread Mike Hommey
On Thu, Jun 08, 2017 at 03:05:37AM -0400, Jeff King wrote: > On Thu, Jun 08, 2017 at 02:34:36PM +0900, Mike Hommey wrote: > > > In 618e613a70, 10 years ago, the default for pack depth used for > > git-pack-objects and git-repack was changed from 10 to 50, while > > leaving fast-import's default

Re: [PATCH] fast-import: Increase the default pack depth to 50

2017-06-08 Thread Jeff King
On Thu, Jun 08, 2017 at 02:34:36PM +0900, Mike Hommey wrote: > In 618e613a70, 10 years ago, the default for pack depth used for > git-pack-objects and git-repack was changed from 10 to 50, while > leaving fast-import's default to 10. > > There doesn't seem to be a reason besides oversight for

[PATCH] fast-import: Increase the default pack depth to 50

2017-06-07 Thread Mike Hommey
In 618e613a70, 10 years ago, the default for pack depth used for git-pack-objects and git-repack was changed from 10 to 50, while leaving fast-import's default to 10. There doesn't seem to be a reason besides oversight for the change not having happened in fast-import as well. Interestingly,