Re: [RFC PATCH 0/5] Add delta islands support

2018-07-26 Thread Johannes Schindelin
Hi Chris, On Sun, 22 Jul 2018, Christian Couder wrote: > This patch series is upstreaming work made by GitHub and available in: > > https://github.com/peff/git/commits/jk/delta-islands > > The patch in the above branch has been split into 5 patches with their > own new commit message, but no o

Re: [RFC PATCH 0/5] Add delta islands support

2018-07-24 Thread Jeff King
On Tue, Jul 24, 2018 at 10:18:03AM -0700, Junio C Hamano wrote: > Jeff King writes: > > >I think this is inherent in the scheme (we're losing some delta > >opportunities). But I think it's also made worse because the delta > >window gets clogged with candidates that are forbidden by

Re: [RFC PATCH 0/5] Add delta islands support

2018-07-24 Thread Junio C Hamano
Jeff King writes: >I think this is inherent in the scheme (we're losing some delta >opportunities). But I think it's also made worse because the delta >window gets clogged with candidates that are forbidden by the island >config. Hmph, and the reason why objects that do not even

Re: [RFC PATCH 0/5] Add delta islands support

2018-07-24 Thread Jeff King
On Sun, Jul 22, 2018 at 07:48:31AM +0200, Christian Couder wrote: > I kept Peff as the author and took the liberty to add his > Signed-off-by to all the patches. Yes, that's fine. This is a topic I've been meaning to upstream for a long time, so I'm happy to see it progressing in that direction.

[RFC PATCH 0/5] Add delta islands support

2018-07-21 Thread Christian Couder
This patch series is upstreaming work made by GitHub and available in: https://github.com/peff/git/commits/jk/delta-islands The patch in the above branch has been split into 5 patches with their own new commit message, but no other change has been made. I kept Peff as the author and took the lib