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

2012-12-10 Thread Lieven Govaerts
Hi Markus, On Mon, Dec 3, 2012 at 10:43 AM, Markus Schaber wrote: > Hi, > > Just another crazy idea: > > The main problem with the parallelization in ra_serf seems to be the number > of http requests (which potentially causes a high number of authentications > and tcp connections). > > Maybe we

Re: non-skelta update editor mode in ra_serf (was Re: svn commit: r1415864 - /subversion/trunk/subversion/libsvn_ra_serf/update.c)

2012-12-05 Thread Lieven Govaerts
Hi, On Sat, Dec 1, 2012 at 10:18 PM, Ivan Zhakov wrote: > [ changing subject to make topic more visible] > > On Sat, Dec 1, 2012 at 9:00 PM, Mark Phippard wrote: >> On Sat, Dec 1, 2012 at 12:36 AM, Justin Erenkrantz >> wrote: >>> On Fri, Nov 30, 2012 at 4:54 PM, wrote: Author: cmpila

AW: svn commit: r1415864 - /subversion/trunk/subversion/libsvn_ra_serf/update.c

2012-12-03 Thread Markus Schaber
HRB 6186 | Tax ID No.: DE 167014915 Von: justin.erenkra...@gmail.com [mailto:justin.erenkra...@gmail.com] Im Auftrag von Justin Erenkrantz Gesendet: Samstag, 1. Dezember 2012 15:15 An: Lieven Govaerts Cc: Johan Corveleyn; dev@subversion.apache.org Betreff: Re: svn commit: r1415864 - /subversion

Re: non-skelta update editor mode in ra_serf (was Re: svn commit: r1415864 - /subversion/trunk/subversion/libsvn_ra_serf/update.c)

2012-12-01 Thread Johan Corveleyn
On Sat, Dec 1, 2012 at 10:40 PM, Branko Čibej wrote: > On 01.12.2012 22:18, Ivan Zhakov wrote: >> Completely agree. >> >> My point was that in theory skelta-mode is cool, but it still needs a >> lot of work to get it really done. > > Sorry? I've had "http-library = serf" in ~/.subversion/config fo

Re: non-skelta update editor mode in ra_serf (was Re: svn commit: r1415864 - /subversion/trunk/subversion/libsvn_ra_serf/update.c)

2012-12-01 Thread Branko Čibej
On 01.12.2012 22:18, Ivan Zhakov wrote: > Completely agree. > > My point was that in theory skelta-mode is cool, but it still needs a > lot of work to get it really done. Sorry? I've had "http-library = serf" in ~/.subversion/config for years, and have seen no problems. What does it take, in your

non-skelta update editor mode in ra_serf (was Re: svn commit: r1415864 - /subversion/trunk/subversion/libsvn_ra_serf/update.c)

2012-12-01 Thread Ivan Zhakov
[ changing subject to make topic more visible] On Sat, Dec 1, 2012 at 9:00 PM, Mark Phippard wrote: > On Sat, Dec 1, 2012 at 12:36 AM, Justin Erenkrantz > wrote: >> On Fri, Nov 30, 2012 at 4:54 PM, wrote: >>> >>> Author: cmpilato >>> Date: Fri Nov 30 21:54:35 2012 >>> New Revision: 1415864 >>>

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

2012-12-01 Thread Justin Erenkrantz
On Sat, Dec 1, 2012 at 12:00 PM, Mark Phippard wrote: > I feel pretty strongly that we should at minimum use the send-all > approach when talking to pre-1.8 servers. Even though in some > situations it could still offer good performance. I just think it > would be more respectful to our users (

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

2012-12-01 Thread Mark Phippard
On Fri, Nov 30, 2012 at 5:38 PM, C. Michael Pilato wrote: > On 11/30/2012 05:25 PM, Mark Phippard wrote: >> On Fri, Nov 30, 2012 at 5:23 PM, C. Michael Pilato >> wrote: >>> On 11/30/2012 05:00 PM, Mark Phippard wrote: On Fri, Nov 30, 2012 at 4:54 PM, wrote: > Author: cmpilato > Da

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

2012-12-01 Thread Mark Phippard
On Sat, Dec 1, 2012 at 12:36 AM, Justin Erenkrantz wrote: > On Fri, Nov 30, 2012 at 4:54 PM, wrote: >> >> Author: cmpilato >> Date: Fri Nov 30 21:54:35 2012 >> New Revision: 1415864 >> >> URL: http://svn.apache.org/viewvc?rev=1415864&view=rev >> Log: >> Implement in ra_serf "send-all" mode suppor

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

2012-12-01 Thread Daniel Shahaf
Justin Erenkrantz wrote on Sat, Dec 01, 2012 at 09:50:29 -0500: > On Sat, Dec 1, 2012 at 9:41 AM, Branko Čibej wrote: > > > On 01.12.2012 14:31, Justin Erenkrantz wrote: > > > And, yes, that clearly could all be done in time for 1.8 without > > > jeopardizing the timelines one tiny bit. =P > > >

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

2012-12-01 Thread Justin Erenkrantz
On Sat, Dec 1, 2012 at 9:41 AM, Branko Čibej wrote: > On 01.12.2012 14:31, Justin Erenkrantz wrote: > > And, yes, that clearly could all be done in time for 1.8 without > > jeopardizing the timelines one tiny bit. =P > > Eep ... :) > > > Another thing I've been thinking about is this: Why are we

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

2012-12-01 Thread Branko Čibej
On 01.12.2012 14:31, Justin Erenkrantz wrote: > And, yes, that clearly could all be done in time for 1.8 without > jeopardizing the timelines one tiny bit. =P Eep ... :) Another thing I've been thinking about is this: Why are we using SHA1 checksums on the server and on the wire for consistency

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

2012-12-01 Thread Justin Erenkrantz
On Sat, Dec 1, 2012 at 9:01 AM, Lieven Govaerts wrote: > There are some scenario's where either the server admin or the user > can decide if parallel requests make sense or not. > > I'm specifically thinking of the use Kerberos per request > authentication. These responses can't be cached on the

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

2012-12-01 Thread Lieven Govaerts
On Sat, Dec 1, 2012 at 2:31 PM, Justin Erenkrantz wrote: > On Sat, Dec 1, 2012 at 5:59 AM, Johan Corveleyn wrote: >> >> I'm wondering whether your concerns apply to both internet-wide >> deployments and local (all on the same LAN) ones. > > > That line is certainly a fair one to draw in the sand.

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

2012-12-01 Thread Justin Erenkrantz
On Sat, Dec 1, 2012 at 5:59 AM, Johan Corveleyn wrote: > I'm wondering whether your concerns apply to both internet-wide > deployments and local (all on the same LAN) ones. > That line is certainly a fair one to draw in the sand. That said, I think the internal use case cries out even *more* fo

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

2012-12-01 Thread Johan Corveleyn
On Sat, Dec 1, 2012 at 6:36 AM, Justin Erenkrantz wrote: > On Fri, Nov 30, 2012 at 4:54 PM, wrote: >> >> Author: cmpilato >> Date: Fri Nov 30 21:54:35 2012 >> New Revision: 1415864 >> >> URL: http://svn.apache.org/viewvc?rev=1415864&view=rev >> Log: >> Implement in ra_serf "send-all" mode support

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

2012-11-30 Thread Daniel Shahaf
On Sat, Dec 01, 2012 at 12:36:30AM -0500, Justin Erenkrantz wrote: > Here's my suggestion for consideration - let's experiment with this setting > in the beta release process with the setting as-is - that is we always do > the parallel updates unconditionally (except perhaps when svnrdump is being

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

2012-11-30 Thread Justin Erenkrantz
On Fri, Nov 30, 2012 at 4:54 PM, wrote: > Author: cmpilato > Date: Fri Nov 30 21:54:35 2012 > New Revision: 1415864 > > URL: http://svn.apache.org/viewvc?rev=1415864&view=rev > Log: > Implement in ra_serf "send-all" mode support for update-style REPORTs > and their responses. (Currently disabled

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

2012-11-30 Thread C. Michael Pilato
On 11/30/2012 05:25 PM, Mark Phippard wrote: > On Fri, Nov 30, 2012 at 5:23 PM, C. Michael Pilato > wrote: >> On 11/30/2012 05:00 PM, Mark Phippard wrote: >>> On Fri, Nov 30, 2012 at 4:54 PM, wrote: Author: cmpilato Date: Fri Nov 30 21:54:35 2012 New Revision: 1415864 U

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

2012-11-30 Thread Mark Phippard
On Fri, Nov 30, 2012 at 5:23 PM, C. Michael Pilato wrote: > On 11/30/2012 05:00 PM, Mark Phippard wrote: >> On Fri, Nov 30, 2012 at 4:54 PM, wrote: >>> Author: cmpilato >>> Date: Fri Nov 30 21:54:35 2012 >>> New Revision: 1415864 >>> >>> URL: http://svn.apache.org/viewvc?rev=1415864&view=rev >>>

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

2012-11-30 Thread C. Michael Pilato
On 11/30/2012 05:00 PM, Mark Phippard wrote: > On Fri, Nov 30, 2012 at 4:54 PM, wrote: >> Author: cmpilato >> Date: Fri Nov 30 21:54:35 2012 >> New Revision: 1415864 >> >> URL: http://svn.apache.org/viewvc?rev=1415864&view=rev >> Log: >> Implement in ra_serf "send-all" mode support for update-sty

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

2012-11-30 Thread Mark Phippard
On Fri, Nov 30, 2012 at 4:54 PM, wrote: > Author: cmpilato > Date: Fri Nov 30 21:54:35 2012 > New Revision: 1415864 > > URL: http://svn.apache.org/viewvc?rev=1415864&view=rev > Log: > Implement in ra_serf "send-all" mode support for update-style REPORTs > and their responses. (Currently disabled