Re: [PATCH] speed up svn_repos_authz_check_access

Ivan Zhakov <ivan_at_visualsvn.com> writes:

> As far I understand cache will be very inefficient if authorization
> file will be bigger than configured limit. I real world cases cache
> will hold data for every configured path if user run svn log on
> repository root or something.

We could cache the results of authz_parse_line. That would limit the
cache size as it would only grow to the number of sections in the file.
Also authz_parse_line is often the expensive bit as it involves checking
the user against all the rules while handling groups and aliases.

When we call svn_config_enumerate2(... authz_parse_line ...) store
section in baton. Then in authz_parse_line use the section from the
baton to check the cache and return the cached result if possible, if
not do the current processing and populate the cache.

-- 
Philip Martin | Subversion Committer
WANdisco | Non-Stop Data