From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Mon, 14 Jul 2008 12:10:47 GMT
Subject: Re: [PATCH 0/4] Honor core.deltaBaseCacheLimit during index-pack
Message-ID: <alpine.DEB.1.00.0807141309340.8950@racer>
In-Reply-To: <m31w1wu1hc.fsf@localhost.localdomain>

Hi,

On Mon, 14 Jul 2008, Jakub Narebski wrote:

> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> > On Mon, 14 Jul 2008, Shawn O. Pearce wrote:
> 
> > > The only other alternative I can come up with is to change pack-objects 
> > > (or at least its defaults) so we don't generate these massive delta 
> > > chains.  But there are already packs in the wild that use these chains, 
> > > resulting in performance problems for clients.
> > 
> > But the long chains make the pack actually as efficient as it is...
> 
> Perhaps Shawn thought here about limiting delta chain not by its
> *length*, but by its *size* (as required when unpacking last object
> in a delta chanin).

So you mean once the sizes of the reconstructed objects are too big, i.e. 
when the delta chain is actually _most useful_, we should not do it?  
Don't think so.

Ciao,
Dscho