Re: svn commit: r1079008 - /subversion/trunk/subversion/svnadmin/main.c

2011-03-13 Thread Stefan Fuhrmann
On 09.03.2011 11:56, Philip Martin wrote: Stefan Fuhrmann writes: On 08.03.2011 14:46, Philip Martin wrote: stef...@apache.org writes: Author: stefan2 Date: Mon Mar 7 22:57:04 2011 New Revision: 1079008 URL: http://svn.apache.org/viewvc?rev=1079008&view=rev Log: Set FSFS cache default siz

Re: svn commit: r1079008 - /subversion/trunk/subversion/svnadmin/main.c

2011-03-09 Thread Philip Martin
Greg Stein writes: > We make changes. We break the code. We make it worse. We fix it, and > move on. Thanks, Greg. I certainly don't want Stefan to feel that I am putting barriers in his way. I probably should have put more effort into getting my questions right. -- Philip

Re: svn commit: r1079008 - /subversion/trunk/subversion/svnadmin/main.c

2011-03-09 Thread Greg Stein
On Tue, Mar 8, 2011 at 11:14, Stefan Sperling wrote: >... > Fair enough, but it should still be opt-in. > We could add a config directive in ~/.subversion/config. > If it turns out to be beneficial, users will enable it. > However, since ra_local is used mostly for testing I don't expect this > to

Re: svn commit: r1079008 - /subversion/trunk/subversion/svnadmin/main.c

2011-03-09 Thread Greg Stein
On ... wrote: >... > Was there any discussion about this overcommit behaviour? This is the second or third time that I've seen "was there any discussion" raised as a point. This raises a small yellow flag for me. We all make changes to the codebase, and many of them are *NOT* discussed before hand

Re: svn commit: r1079008 - /subversion/trunk/subversion/svnadmin/main.c

2011-03-09 Thread Philip Martin
Stefan Fuhrmann writes: > On 08.03.2011 14:46, Philip Martin wrote: >> stef...@apache.org writes: >> >>> Author: stefan2 >>> Date: Mon Mar 7 22:57:04 2011 >>> New Revision: 1079008 >>> >>> URL: http://svn.apache.org/viewvc?rev=1079008&view=rev >>> Log: >>> Set FSFS cache default size to 16 MB. T

Re: svn commit: r1079008 - /subversion/trunk/subversion/svnadmin/main.c

2011-03-08 Thread Stefan Fuhrmann
On 08.03.2011 15:01, Stefan Sperling wrote: On Tue, Mar 08, 2011 at 01:46:23PM +, Philip Martin wrote: stef...@apache.org writes: Author: stefan2 Date: Mon Mar 7 22:57:04 2011 New Revision: 1079008 URL: http://svn.apache.org/viewvc?rev=1079008&view=rev Log: Set FSFS cache default size to

Re: svn commit: r1079008 - /subversion/trunk/subversion/svnadmin/main.c

2011-03-08 Thread Stefan Fuhrmann
On 08.03.2011 14:46, Philip Martin wrote: stef...@apache.org writes: Author: stefan2 Date: Mon Mar 7 22:57:04 2011 New Revision: 1079008 URL: http://svn.apache.org/viewvc?rev=1079008&view=rev Log: Set FSFS cache default size to 16 MB. This is the same default as for everybody else, namely the

Re: svn commit: r1079008 - /subversion/trunk/subversion/svnadmin/main.c

2011-03-08 Thread Philip Martin
Stefan Sperling writes: > On Tue, Mar 08, 2011 at 02:15:06PM +, Philip Martin wrote: >> Stefan Sperling writes: >> >> > So I think it would be better if the cache was off by default, and must be >> > enabled in a configuration file. I suppose svnserve and mod_dav_svn >> > configuration file

Re: svn commit: r1079008 - /subversion/trunk/subversion/svnadmin/main.c

2011-03-08 Thread Stefan Sperling
On Tue, Mar 08, 2011 at 02:15:06PM +, Philip Martin wrote: > Stefan Sperling writes: > > > So I think it would be better if the cache was off by default, and must be > > enabled in a configuration file. I suppose svnserve and mod_dav_svn > > configuration files would be the best location for

RE: svn commit: r1079008 - /subversion/trunk/subversion/svnadmin/main.c

2011-03-08 Thread Bert Huijben
> -Original Message- > From: Ivan Zhakov [mailto:i...@visualsvn.com] > Sent: dinsdag 8 maart 2011 15:08 > To: Philip Martin; dev@subversion.apache.org > Cc: Stefan Sperling > Subject: Re: svn commit: r1079008 - > /subversion/trunk/subversion/svnadmin/main.c > >

Re: svn commit: r1079008 - /subversion/trunk/subversion/svnadmin/main.c

2011-03-08 Thread Philip Martin
Stefan Sperling writes: > So I think it would be better if the cache was off by default, and must be > enabled in a configuration file. I suppose svnserve and mod_dav_svn > configuration files would be the best location for this, svnserve doesn't have a configuration file. There is a per-reposi

Re: svn commit: r1079008 - /subversion/trunk/subversion/svnadmin/main.c

2011-03-08 Thread Ivan Zhakov
On Tue, Mar 8, 2011 at 17:01, Stefan Sperling wrote: > On Tue, Mar 08, 2011 at 01:46:23PM +, Philip Martin wrote: >> stef...@apache.org writes: >> >> > Author: stefan2 >> > Date: Mon Mar  7 22:57:04 2011 >> > New Revision: 1079008 >> > >> > URL: http://svn.apache.org/viewvc?rev=1079008&view=re

Re: svn commit: r1079008 - /subversion/trunk/subversion/svnadmin/main.c

2011-03-08 Thread Stefan Sperling
On Tue, Mar 08, 2011 at 01:46:23PM +, Philip Martin wrote: > stef...@apache.org writes: > > > Author: stefan2 > > Date: Mon Mar 7 22:57:04 2011 > > New Revision: 1079008 > > > > URL: http://svn.apache.org/viewvc?rev=1079008&view=rev > > Log: > > Set FSFS cache default size to 16 MB. This is t

Re: svn commit: r1079008 - /subversion/trunk/subversion/svnadmin/main.c

2011-03-08 Thread Philip Martin
stef...@apache.org writes: > Author: stefan2 > Date: Mon Mar 7 22:57:04 2011 > New Revision: 1079008 > > URL: http://svn.apache.org/viewvc?rev=1079008&view=rev > Log: > Set FSFS cache default size to 16 MB. This is the same default as > for everybody else, namely the server processes. Thus, it sh