Re: [PATCH 0/2] allow git-svn fetching to work using serf

2013-07-07 Thread Jonathan Nieder
(cc-ing users@ as requested by danielsh) David Rothenberger wrote: > On 7/6/2013 5:28 PM, Jonathan Nieder wrote: >> Is there a simple explanation of why violating the depth-first >> constraint would lead to multiple blob (i.e., file, not directory) >> deltas being opened in a row without an interv

Re: [PATCH 0/2] allow git-svn fetching to work using serf

2013-07-07 Thread Jonathan Nieder
(cc-ing subversion's users@ list for advice) Kyle McKay wrote: > On Jul 6, 2013, at 18:37, Jonathan Nieder wrote: >> Kyle McKay wrote: >>> Begin forwarded message: [2] http://subversion.tigris.org/issues/show_bug.cgi?id=2932 >> >> Ah, thanks for the context. >> >> It's still not clear to me h

Re: [PATCH 0/2] allow git-svn fetching to work using serf

2013-07-06 Thread David Rothenberger
On 7/6/2013 5:28 PM, Jonathan Nieder wrote: > David Rothenberger wrote: >> On 7/5/2013 8:41 PM, Kyle McKay wrote: > >>> Daniel Shahaf has suggested also setting >>> "servers:global:http-bulk-updates=on". >> >> I have a patch that does this, but since turning on bulk updates has >> a possible perfo

Re: [PATCH 0/2] allow git-svn fetching to work using serf

2013-07-06 Thread Kyle McKay
On Jul 6, 2013, at 18:37, Jonathan Nieder wrote: Kyle McKay wrote: On Jul 6, 2013, at 17:28, Jonathan Nieder wrote: David Rothenberger wrote: On 7/5/2013 8:41 PM, Kyle McKay wrote: Daniel Shahaf has suggested also setting "servers:global:http-bulk-updates=on". I have a patch that does thi

Re: [PATCH 0/2] allow git-svn fetching to work using serf

2013-07-06 Thread Jonathan Nieder
Kyle McKay wrote: > On Jul 6, 2013, at 17:28, Jonathan Nieder wrote: >> David Rothenberger wrote: >>> On 7/5/2013 8:41 PM, Kyle McKay wrote: Daniel Shahaf has suggested also setting "servers:global:http-bulk-updates=on". >>> >>> I have a patch that does this, but since turning on bulk up

Re: [PATCH 0/2] allow git-svn fetching to work using serf

2013-07-06 Thread Kyle McKay
On Jul 6, 2013, at 17:28, Jonathan Nieder wrote: David Rothenberger wrote: On 7/5/2013 8:41 PM, Kyle McKay wrote: Daniel Shahaf has suggested also setting "servers:global:http-bulk-updates=on". I have a patch that does this, but since turning on bulk updates has a possible performance penal

Re: [PATCH 0/2] allow git-svn fetching to work using serf

2013-07-06 Thread Jonathan Nieder
David Rothenberger wrote: > On 7/5/2013 8:41 PM, Kyle McKay wrote: >> Daniel Shahaf has suggested also setting >> "servers:global:http-bulk-updates=on". > > I have a patch that does this, but since turning on bulk updates has > a possible performance penalty, I prefer your approach. I assume tha

Re: [PATCH 0/2] allow git-svn fetching to work using serf

2013-07-06 Thread David Rothenberger
On 7/5/2013 8:41 PM, Kyle McKay wrote: > This patch allows git-svn to fetch successfully using the > serf library when given an https?: url to fetch from. Thanks, Kyle. I confirm this is working for my problem cases as well. > Daniel Shahaf has suggested also setting > "servers:global:http-bulk-u

[PATCH 0/2] allow git-svn fetching to work using serf

2013-07-05 Thread Kyle McKay
This patch allows git-svn to fetch successfully using the serf library when given an https?: url to fetch from. Unfortunately some svn servers do not seem to be configured well for use with the serf library. This can cause fetching to take longer compared to the neon library or actually cause ti