Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

On Tue, Jun 25, 2013 at 6:21 PM, Ivan Zhakov <ivan_at_visualsvn.com> wrote:
> On Wed, Jun 26, 2013 at 2:12 AM, Daniel Shahaf <danielsh_at_elego.de> wrote:
>> Johan Corveleyn wrote on Tue, Jun 25, 2013 at 23:22:12 +0200:
>...
>>> The dynamic detection has a cost (1 extra request per connection),
>>> that you might want to avoid by default (most environments won't need
>>> the dynamic detection (especially corporate environments)). Only
>>> enable the dynamic detection if you know the proxy has a problem with
>>> chunkness, or if you're not sure it will stay that way, or ...

Right. My intent is a config option to enable the extra request when
problems may be present.

>...
>> AIUI the cost is only incurred by set-ups that have the so-called
>> "busted" proxies. And a config option has a cost too: it would need to
>> be supported until 2.0 (aka, indefinitely).

Right.

> Please note that this extra request is per session and currently we
> create many sessions even during one operation. And I'm also not happy
> to make performance worse for users who doesn't use reverse proxies
> and etc.

People with busted proxies will just need to deal with it. And
libsvn_client should work on using fewer sessions.

Most people will never enable the config, and never have a problem or
reduced performance.

Cheers,
-g
Received on 2013-06-26 00:26:16 CEST