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).
