Re: [PATCH] speed up svn_repos_authz_check_access

On Wed, Aug 7, 2013 at 5:43 PM, Roderich Schupp
<roderich.schupp_at_gmail.com> wrote:
> Hi,
>
> revised patch attached, hopefully addressing all concerns.
>
> No more breaking encapsulation. We simply allocate svn_authz_t from the same
> pool
> that svn_config_t is allocated from and remember this pool in a new
> svn_authz_t
> member "pool".l Create a function authz_create() for that; it uses
> apr_pcalloc
> so that we don't have to explicitly zero members "cache" and "cached_user"
> any more.
>
Could you please provide log message for your patch submission [1]. It
makes reviewing patch much easier.

Did you consider make separate cache_pool and clear if we invoked with
different user?

[1] http://subversion.apache.org/docs/community-guide/general.html#patches

-- 
Ivan Zhakov
CTO | VisualSVN | http://www.visualsvn.com