[PATCH 1 of 4] manifest: make lru size configurable

Matt Mackall mpm at selenic.com
Tue Feb 3 18:19:17 CST 2015


On Tue, 2015-02-03 at 15:20 +0000, Pierre-Yves David wrote:
> 
> On 02/03/2015 03:19 AM, Durham Goode wrote:
> > # HG changeset patch
> > # User Durham Goode <durham at fb.com>
> > # Date 1422061563 28800
> > #      Fri Jan 23 17:06:03 2015 -0800
> > # Node ID 83bf8c0ecb425fb64903d93918227c162bca111d
> > # Parent  8b88870cbd1eeefaee0af053ae36728f8c0a1847
> > manifest: make lru size configurable
> >
> > On machines with lots of ram, it's beneficial to increase the lru size of the
> > manifest cache.  On a large repo, configuring the lru to be size 10 can shave a
> > large rebase (~12 commits) down from 95s to 70s.
> 
> The option for this appears to be
> 
> 'ui.manifestcachesize' this is hardly a 'ui' option. We currently do not 
> have a good section to fit this. (not really format, not really ui). 
> maybe we could rename the `workers` section to `ressources` or 
> `performances` or `tuning`. And put them both there.

We have precedent in format.chunkcachesize, so format seems the obvious
place.

You might be interested in investigating the performance impact of that
variable, btw.

-- 
Mathematics is the supreme nostalgia of our time.




More information about the Mercurial-devel mailing list