Re: serf in 1.8

2012-11-13 Thread Lieven Govaerts
Ivan, On Tue, Nov 13, 2012 at 11:00 AM, Ivan Zhakov wrote: > On Tue, Nov 13, 2012 at 11:26 AM, Lieven Govaerts wrote: >> On Tue, Nov 13, 2012 at 12:20 AM, Philip Martin > [...] > >> Also, there might well be a windows-kerberos triple trusted >> domain .. setup somewhere where svn+neon works and

Re: Double compression over HTTPS

2012-11-13 Thread Lieven Govaerts
On Tue, Nov 13, 2012 at 8:41 AM, Lieven Govaerts wrote: > On Fri, Oct 5, 2012 at 11:51 AM, Lieven Govaerts wrote: >> Hi, >> >> >> >> when OpenSSL is built with zlib, it will automatically compress all data >> sent over an SSL connection. You can see this in the initial handshake >> "Client Hello"

Re: fork/exec for hooks scripts with a large FSFS cache

2012-11-13 Thread Greg Stein
This problem is why mod_cgid exists. Been there. Done that. It creates a daemon early, before threads/mem are allocated. The child CGI processes are then fast to spawn. Cheers, -g (*) http://httpd.apache.org/docs/2.4/mod/mod_cgid.html On Nov 13, 2012 6:44 PM, "Daniel Shahaf" wrote: > Philip Mar

Re: fork/exec for hooks scripts with a large FSFS cache

2012-11-13 Thread Daniel Shahaf
Philip Martin wrote on Tue, Nov 13, 2012 at 21:30:00 +: > Perhaps we could start up a separate hook script process before > allocating the large FSFS cache and then delegate the fork/exec to that > smaller process? If so, let's have that daemon handle all forks we might do, not just those rela

Re: fork/exec for hooks scripts with a large FSFS cache

2012-11-13 Thread Stefan Fuhrmann
On Tue, Nov 13, 2012 at 10:30 PM, Philip Martin wrote: > If I configure a server with an FSFS cache that uses about 50% of > available memory and then I use the server so the cache is in use I find > that hook scripts fail to run because the fork/exec cannot allocate > memory. The user sees: > I

Re: [Issue 3980] serf increases server load

2012-11-13 Thread Stefan Fuhrmann
On Mon, Nov 12, 2012 at 9:37 PM, Philip Martin wrote: > Stefan Fuhrmann writes: > > >> > Using serf 1.1.x@1691 and subversion trunk@1408335 as 1.8. > >> > Using 1.7.7 with neon as 1.7. > >> > Using subversion trunk as my dataset. > >> > > >> > The server CPU and bandwidth to service one checkout:

Re: serf in 1.8

2012-11-13 Thread kmradke
> From: Philip Martin > kmra...@rockwellcollins.com writes: > > > *one* of my servers currently logs around 10GB per week with neon > > only access. I'd probably have to change from rotating logs > > weekly to rotating logs hourly! Disabling logging is not an option. > > It depends what sort o

fork/exec for hooks scripts with a large FSFS cache

2012-11-13 Thread Philip Martin
If I configure a server with an FSFS cache that uses about 50% of available memory and then I use the server so the cache is in use I find that hook scripts fail to run because the fork/exec cannot allocate memory. The user sees: $ svn mkdir -mm http://localhost:/obj/repo/A svn: E165002: Faile

Re: serf in 1.8

2012-11-13 Thread Philip Martin
kmra...@rockwellcollins.com writes: > *one* of my servers currently logs around 10GB per week with neon > only access. I'd probably have to change from rotating logs > weekly to rotating logs hourly! Disabling logging is not an option. It depends what sort of access patterns you see. The probl

Re: serf in 1.8

2012-11-13 Thread Stefan Küng
On 13.11.2012 21:03, Mark Phippard wrote: On Tue, Nov 13, 2012 at 2:48 PM, Mark Phippard wrote: On Tue, Nov 13, 2012 at 2:44 PM, Stefan Küng wrote: On 13.11.2012 14:58, Mark Phippard wrote: We did some testing in our lab, and the KeepAlive settings help a lot here. Without any KeepAlive, t

Re: serf in 1.8

2012-11-13 Thread Stefan Küng
On 13.11.2012 20:48, Mark Phippard wrote: On Tue, Nov 13, 2012 at 2:44 PM, Stefan Küng wrote: On 13.11.2012 14:58, Mark Phippard wrote: We did some testing in our lab, and the KeepAlive settings help a lot here. Without any KeepAlive, then obviously every Serf request on every connection nee

Re: serf in 1.8

2012-11-13 Thread Mark Phippard
On Tue, Nov 13, 2012 at 2:48 PM, Mark Phippard wrote: > On Tue, Nov 13, 2012 at 2:44 PM, Stefan Küng wrote: >> On 13.11.2012 14:58, Mark Phippard wrote: >> >>> We did some testing in our lab, and the KeepAlive settings help a lot >>> here. Without any KeepAlive, then obviously every Serf request

Re: serf in 1.8

2012-11-13 Thread Mark Phippard
On Tue, Nov 13, 2012 at 2:44 PM, Stefan Küng wrote: > On 13.11.2012 14:58, Mark Phippard wrote: > >> We did some testing in our lab, and the KeepAlive settings help a lot >> here. Without any KeepAlive, then obviously every Serf request on >> every connection needed to be re-authenticated. With

Re: serf in 1.8

2012-11-13 Thread Stefan Küng
On 13.11.2012 14:58, Mark Phippard wrote: We did some testing in our lab, and the KeepAlive settings help a lot here. Without any KeepAlive, then obviously every Serf request on every connection needed to be re-authenticated. With KeepAlive on and the connection limit set high enough then it w

Re: svn commit: r1408849 - /subversion/trunk/subversion/libsvn_ra_serf/update.c

2012-11-13 Thread Stefan Sperling
On Tue, Nov 13, 2012 at 02:33:09PM -0500, C. Michael Pilato wrote: > On 11/13/2012 02:21 PM, Stefan Sperling wrote: > > Are you sure this is not a bug checking the wrong X or setting the wrong > > X to TRUE? I don't know ra_serf very well so this isn't obvious to me. > > I'm pretty sure it was a b

Re: svn commit: r1408849 - /subversion/trunk/subversion/libsvn_ra_serf/update.c

2012-11-13 Thread C. Michael Pilato
On 11/13/2012 02:21 PM, Stefan Sperling wrote: > On Tue, Nov 13, 2012 at 05:32:00PM -, cmpil...@apache.org wrote: >> Author: cmpilato >> Date: Tue Nov 13 17:31:59 2012 >> New Revision: 1408849 >> >> URL: http://svn.apache.org/viewvc?rev=1408849&view=rev >> Log: >> * subversion/libsvn_ra_serf/up

Re: svn commit: r1408849 - /subversion/trunk/subversion/libsvn_ra_serf/update.c

2012-11-13 Thread Stefan Sperling
On Tue, Nov 13, 2012 at 05:32:00PM -, cmpil...@apache.org wrote: > Author: cmpilato > Date: Tue Nov 13 17:31:59 2012 > New Revision: 1408849 > > URL: http://svn.apache.org/viewvc?rev=1408849&view=rev > Log: > * subversion/libsvn_ra_serf/update.c > (end_report): Remove redundant boolean assig

Re: svn commit: r1408325 - /subversion/branches/wc-collate-path/subversion/libsvn_subr/sqlite.c

2012-11-13 Thread Greg Stein
On Nov 13, 2012 4:00 AM, "Branko Čibej" wrote: >... > extract a detailed design. In the meantime, if it makes you feel better, > you can treat that branch as my private playground that's never intended > to be merged back to trunk as-is. Could you add a branch readme, stating such? I think that w

Re: serf in 1.8

2012-11-13 Thread kmradke
> Greg Stein writes: > > > On Mon, Nov 12, 2012 at 6:20 PM, Philip Martin > > wrote: > >>... > >> Another concern is the increased server logging due to the large > >> increase in the number of requests. A 1.8 server does better than older > >> servers, about 50% fewer requests on checkout, bu

Re: serf in 1.8

2012-11-13 Thread Mark Phippard
On Mon, Nov 12, 2012 at 6:20 PM, Philip Martin wrote: > We at WANdisco had a discussion about serf in 1.8 today; I'd said I > summarise to the list. > > One concern is the impact on server performance, particularly older > servers, of serf as the only client. Issue > http://subversion.tigris.org/

Re: [PATCH]: Encoding fix mailer.py

2012-11-13 Thread C. Michael Pilato
On 11/13/2012 03:19 AM, Igor Galić wrote: >>> >>> Igor, I took a look at your patch. I was confused by one aspect: >>> why split >>> the from_addr into words an encode each one individually, rather >>> than >>> encoding the whole header value like we do for the Subject: header? >>> I dug >>> aroun

Re: serf in 1.8

2012-11-13 Thread Ivan Zhakov
On Tue, Nov 13, 2012 at 4:47 PM, Philip Martin wrote: > Justin Erenkrantz writes: > >> On Tue, Nov 13, 2012 at 2:26 AM, Lieven Govaerts wrote: >> >>> Is that a fix that we can get in apache 2.2? Justin? >>> >> >> I don't readily recall the issue in mod_deflate. Is it already resolved in >> the

Re: [Issue 2625] Better fix for convoluted installations.(where 3rd party libs collocated in old version of svn)

2012-11-13 Thread C. Michael Pilato
On 11/13/2012 03:41 AM, danie...@tigris.org wrote: > http://subversion.tigris.org/issues/show_bug.cgi?id=2625 > > > > > > > --- Additional comments from danie...@tigris.org Tue Nov 13 00:41:51 > -0800 2012 --- > Six years later... > > Neon has been removed, Serf replaced it, and bui

RE: [PATCH] Implement '--include-externals' option to 'svn list'

2012-11-13 Thread Bert Huijben
> -Original Message- > From: vijay [mailto:vi...@collab.net] > Sent: dinsdag 13 november 2012 14:06 > To: Subversion Development > Subject: Re: [PATCH] Implement '--include-externals' option to 'svn list' > > On Tuesday 13 November 2012 03:02 AM, Stefan Sperling wrote: > >> Attached the

Re: [PATCH] Implement '--include-externals' option to 'svn list'

2012-11-13 Thread vijay
On Tuesday 13 November 2012 03:02 AM, Stefan Sperling wrote: Attached the updated patch and log message. + /* Notify that we're about to handle an external. */ + SVN_ERR(list_func(baton, NULL, NULL, NULL, NULL, +externals_parent_url, +i

Re: serf in 1.8

2012-11-13 Thread Philip Martin
Justin Erenkrantz writes: > On Tue, Nov 13, 2012 at 2:26 AM, Lieven Govaerts wrote: > >> Is that a fix that we can get in apache 2.2? Justin? >> > > I don't readily recall the issue in mod_deflate. Is it already resolved in > the 2.4 series? r1103315 You wrote about it: http://mail-archives.

Re: serf in 1.8

2012-11-13 Thread Justin Erenkrantz
On Tue, Nov 13, 2012 at 2:26 AM, Lieven Govaerts wrote: > Is that a fix that we can get in apache 2.2? Justin? > I don't readily recall the issue in mod_deflate. Is it already resolved in the 2.4 series? > work. These concern maybe only 1% of the users, but we should have a > plan to support

Re: svn commit: r1408686 - /subversion/trunk/subversion/svnmucc/svnmucc.c

2012-11-13 Thread Daniel Shahaf
rhuij...@apache.org wrote on Tue, Nov 13, 2012 at 11:57:02 -: > Author: rhuijben > Date: Tue Nov 13 11:57:01 2012 > New Revision: 1408686 > > URL: http://svn.apache.org/viewvc?rev=1408686&view=rev > Log: > Apply some correctness and debug easing changes on svnmucc. > > @@ -792,8 +785,10 @@ ex

Re: [serf-dev] Re: Double compression over HTTPS

2012-11-13 Thread Justin Erenkrantz
On Tue, Nov 13, 2012 at 2:41 AM, Lieven Govaerts wrote: > The proposed approach here is to disable SSL compression completely, > so in terms of the above that leaves us with scenario 2. > > Serf doesn't have an option currently to disable SSL compression from > the client side. I plan to add it i

Re: svn commit: r1408325 - /subversion/branches/wc-collate-path/subversion/libsvn_subr/sqlite.c

2012-11-13 Thread Branko Čibej
On 12.11.2012 18:34, Bert Huijben wrote: >> -Original Message- >> From: Branko Čibej [mailto:br...@wandisco.com] >> Sent: maandag 12 november 2012 17:49 >> To: dev@subversion.apache.org >> Subject: Re: svn commit: r1408325 - /subversion/branches/wc-collate- >> path/subversion/libsvn_subr/sq

Re: serf in 1.8

2012-11-13 Thread Ivan Zhakov
On Tue, Nov 13, 2012 at 11:26 AM, Lieven Govaerts wrote: > On Tue, Nov 13, 2012 at 12:20 AM, Philip Martin [...] > Also, there might well be a windows-kerberos triple trusted > domain .. setup somewhere where svn+neon works and svn+serf doesn't > work. What is "windows-kerberos tripli trusted dom

Re: [serf-dev] Re: Double compression over HTTPS

2012-11-13 Thread Ivan Zhakov
On Tue, Nov 13, 2012 at 11:41 AM, Lieven Govaerts wrote: > On Fri, Oct 5, 2012 at 11:51 AM, Lieven Govaerts wrote: >> Hi, >> when OpenSSL is built with zlib, it will automatically compress all data >> sent over an SSL connection. You can see this in the initial handshake >> "Client Hello" and "Se

Re: [PATCH]: Encoding fix mailer.py

2012-11-13 Thread Igor Galić
> > > > Igor, I took a look at your patch. I was confused by one aspect: > > why split > > the from_addr into words an encode each one individually, rather > > than > > encoding the whole header value like we do for the Subject: header? > > I dug > > around in the related RFCs, and if I'm reading