Re: [PATCH 0/5] Suggested for PU: revision caching system to significantly speed up packing/walking
- From
Shawn O. Pearce <spearce@spearce.org>
- Date
- Aug 6, 2009, 20:32 UTC
- Message-ID
- <20090806203223.GK1033@spearce.org>
- In-Reply-To
- <c77435a80908061330h2461012at8b877970cab4906b@mail.gmail.com>
Nick Edelen <sirnot@gmail.com> wrote:
Show 8 quoted lines
> Hrmm, I just realized that it dosn't actually cache paths/names... > This obviously has no bearing on its use in packing, but I should > either add that in or restrict usage in non-packing-related walks. > Weird how things like that escape you. > > I think I may go ahead and add support for this tomorrow. It should > have no effect on performance and very little impact on cache slice > size.
You may not need the path name, but instead the hash value that pack-objects computes from the path name. All that matters is the hash, so pack-objects can schedule the objects into the right buckets when its doing delta computation for objects which are not yet delta compressed, or whose delta cannot be suitably reused.
-- Shawn.