Re: [PATCH] speed up svn_repos_authz_check_access

On Wed, Aug 7, 2013 at 1:02 PM, Daniel Shahaf <danielsh_at_elego.de> wrote:
> Please use text/* MIME type. The .c file was so but the .patch wasn't,
> and that gets in the way of reviewing it. More below...
>
> Roderich Schupp wrote on Wed, Aug 07, 2013 at 08:27:43 +0200:
>> This patch reduces the cumulative time for svn_repos_authz_check_access
>> (when called repeteadly in the same connection) for more than 50%
>> (e.g. running the attached test program on a large body of paths).
>>
>
> Can this break anything?
>
> For example, when a client uses a long-lived keep-alive connection and
> the authz file changes during the life of that connection, will your
> change cause the changes to be picked up later than they would before
> your patch?
>
We already cache authz in memory for connection. So this is not issue.

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