[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1344347 - /subversion/trunk/subversion/libsvn_subr/config_file.c

From: Ivan Zhakov <ivan_at_visualsvn.com>
Date: Thu, 31 May 2012 13:47:42 +0400

On Thu, May 31, 2012 at 1:27 PM, Greg Stein <gstein_at_gmail.com> wrote:
> On Thu, May 31, 2012 at 4:39 AM, Philip Martin
> <philip.martin_at_wandisco.com> wrote:
>> Ivan Zhakov <ivan_at_visualsvn.com> writes:
>>
>>> Config files are also used for authz settings and they can be even
>>> more than 100 MB in real world scenarios.
>>
>> Yes, particularly for setups that use SVNParentPath but not the new
>> AuthzSVNReposRelativeAccessFile.
>
> Woah. Wait a second here.
>
> My understanding is that these are read on *every* request (per
> Daniel). Are you suggesting that we are reading/parsing 100 Mb on
> every request?
>
> There is something wrong here. 100 Mb config files are flat out wrong.
> We should be doing better. And if we are *actually* parsing those
> per-request, then we've gone off the deep-end.
>
> ???
Currently Subversion reads authz file per-connection, not per-request.

-- 
Ivan Zhakov
Received on 2012-05-31 11:48:37 CEST

This is an archived mail posted to the Subversion Dev mailing list.