Re: [PATCH 0/4] Honor core.deltaBaseCacheLimit during index-pack
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Jul 14, 2008, 12:10 UTC
- 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:
Show 13 quoted lines
> 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