Re: Optimizing back-end: saving cycles or saving I/O (was: Re: [PATCH v2] Saving a few cycles, part 3/3)

Johan Corveleyn wrote on Wed, 12 May 2010 at 01:16 +0200:
> Yes, some form of caching may or may not be a good approach, but the
> main point is that ideally, for a certain client request, every
> interesting rev file should be opened and read exactly once. Currently
> this is definitely not the case (for "svn log" it's closer to 10
> opens/closes and 5 times the amount of bytes of every rev file
> involved;

/me nods

> with packed revs it's even worse because of the extra lookup
> of the rev offset in the pack manifest file).
>

The offsets are cached now (e.g., ffd->packed_offset_cache).
Received on 2010-05-12 07:05:28 CEST