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

RE: svn commit: r1725184 - /subversion/branches/1.9.x/STATUS

From: Bert Huijben <bert_at_qqmail.nl>
Date: Mon, 18 Jan 2016 10:47:12 +0100

> -----Original Message-----
> From: stefan2_at_apache.org [mailto:stefan2_at_apache.org]
> Sent: maandag 18 januari 2016 09:22
> To: commits_at_subversion.apache.org
> Subject: svn commit: r1725184 - /subversion/branches/1.9.x/STATUS
>
> Author: stefan2
> Date: Mon Jan 18 08:22:27 2016
> New Revision: 1725184
>
> URL: http://svn.apache.org/viewvc?rev=1725184&view=rev
> Log:
> * STATUS:
> Add r1721285 (raise size limit on cachable directories).
>

<snip>

> + * r1721285
> + Quadruple the maximum cacheable directory size in FSFS.
> + Justification:
> + Not caching large directories has a massive impact on runtime with
> + the cutoff point being about 9000 entries for the default cache size.
> + Also, memory usage with mod_dav_svn may go through the roof for
> + uncached directories.
> + Votes:
> + +1: stefan2

When I look at the patch (and the log message), it looks at a very simple change... but I have absolutely no idea what the side effects are.

Given your descriptions, I would like to see it backported, but can you help us review this change?

Thanks,
        Bert
Received on 2016-01-18 10:47:33 CET

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.