[MERGE] Use a LRUCache in LocalTransport.clone

Andrew Bennetts andrew at canonical.com
Tue Jan 15 00:34:59 GMT 2008


Robert Collins wrote:
> 
> On Sat, 2008-01-05 at 01:41 +1100, Andrew Bennetts wrote:
> > This is a simple change to use a LRUCache in LocalTransport.clone.  It seems
> > that calculating an abspath can be quite expensive.  By keeping the LRUCache I
> > reduce the time to pull a 300 revisions from a knit repo into a pack repo from
> > ~32 seconds to ~26 seconds!
> 
> Thats nice, but why do we care? We're trying to migrate to packs and
> conversions should be one-offs. It should not be any slower than knit to
> knit cloning anyhow.

FWIW, I noticed this because I was branching from knit repo into a pack repo,
and was curious about why it was so slow.  This isn't a common operation for me,
seeing as I use packs almost everywhere now, so I'm not too worried if this
speed-up lands or not.

I did think it was interesting enough to share with the list, though.

-Andrew.




More information about the bazaar mailing list