[PATCH 3 of 8] Create repository objects instead of using own ConfigParser in hgwebdir

Thomas Arendsen Hein thomas at intevation.de
Sun Aug 28 23:42:53 CDT 2005


* Matt Mackall <mpm at selenic.com> [20050828 22:34]:
> On Sun, Aug 28, 2005 at 06:54:53PM +0200, Thomas Arendsen Hein wrote:
> > # HG changeset patch
> > # User Thomas Arendsen Hein <thomas at intevation.de>
> > # Node ID a67982e64109087eb728b5cb7a45a37c5785639b
> > # Parent  1ad52c7679e14abd40f6f3e319f707df63bc6daa
> > Create repository objects instead of using own ConfigParser in hgwebdir.
> 
> This has some performance implications, which is why I didn't do it
> the first time around. Opening a repo requires parsing the changelog
> and manifest indices. It takes a non-negligible amount of time to do
> this (looks to be about .007s on my box) so opening a dozen repos
> every time someone hits the index page is not very desirable.

Ok, as you convinced me to show the mtime of 00changelog.d on IRC we
no longer need this.

But we still should use the ui object, because this gives much more
consistent access to the hgrc file, its global (/etc and ~)
counterparts, and in the future it will solve encoding issues.

This is a bit slower than the original, but still much faster than
creating real repo objects.

I'll rewrite this and send a new patchbomb.

Thomas

-- 
Email: thomas at intevation.de
http://intevation.de/~thomas/


More information about the Mercurial mailing list