Re: [PATCH] speed up svn_repos_authz_check_access

On Sun, Aug 11, 2013 at 11:17 PM, Philip Martin
<philip.martin_at_wandisco.com>wrote:

> To see the difference consider an authz file with just one block for /.
> Given two paths, /foo/bar and /zig/zag, your patch produces two cache
> entries. Repeat requests for one of the paths pull the result directly
> from the cache. The cache grows without limit as further distinct paths
> are requested.
>

That's not how my patch works (the description could probably be more
explicit
about this aspect): we only cache the "result" of svn_config_enumerate2
if the given path corresponds to a block (section) in the authz file.
(Otherwise we don't call svn_config_enumerate2 at all and cache nothing.)
Hence the cache is limited by the size of the authz file (i.e. keys of
apt_hash_t svn_config_t->sections.

Cheers, Roderich
Received on 2013-08-12 09:55:46 CEST