I think Lieven is talking about three timeout values:
1) serf context loop, set to 500ms to 1000ms, maybe.
2) iniitial response from server, set to 60 seconds?
3) continuing response from server, set to 5 seconds?
The first timeout gives us our responsiveness for a GUI. As Lieven
mentions in his reply, we can also check for cancellation in the
response handler.
The second timeout is one that Johan was concerned about: long hook
processing that delays the initial response.
The third timeout is a normal kind of networking timeout, where it may
appear the connection has dropped.
We can debate these values, but I believe the short answer is that we
can solve the cancellation problem.
Cheers,
-g
On Mon, May 21, 2012 at 1:33 PM, Bert Huijben <bert_at_qqmail.nl> wrote:
> Even with a sensible timeout of 20/30 seconds instead of half an hour, that
> value would still be much too high for blocking a GUI tool. The cancel
> problem must also be fixed.
>
>
>
> You press ‘Cancel’ in the TortoiseSVN dialog…. And have to wait for 30
> minutes until it is processed.
>
>
>
> On *nix you don’t have this problem for handling ^C, as that breaks IO, but
> in GUIs that doesn’t work.
>
>
>
> Bert
>
>
>
>
>
> From: lieven.govaerts_at_gmail.com [mailto:lieven.govaerts_at_gmail.com] On Behalf
> Of Lieven Govaerts
> Sent: maandag 21 mei 2012 19:28
> To: Johan Corveleyn; dev_at_subversion.apache.org
> Cc: Greg Stein
>
>
> Subject: Re: [DISCUSS] delete ra_neon
>
>
>
>
>
> On Mon, May 21, 2012 at 1:47 PM, Johan Corveleyn <jcorvel_at_gmail.com> wrote:
>
> On Sat, May 19, 2012 at 9:58 PM, Greg Stein <gstein_at_gmail.com> wrote:
>> So far, beyond Philip's concern about traffic/CPU tradeoff, and a
>> couple open issues... I think we're in good shape to pull the lever.
>
> As I said elsethread, this is probably the best overview of open
> ra_serf blockers:
>
> http://subversion.tigris.org/issues/buglist.cgi?subcomponent=libsvn_ra_serf&issue_status=NEW&issue_status=STARTED&issue_status=REOPENED&target_milestone=1.8.0
>
> For convenience, I'm pasting the current result of this query here:
>
> 3968: DEFAULT_HTTP_TIMEOUT is way too high for serf
> 3969: ra_serf should check for cancellation from loops around
> serf_context_run()
>
>
>
> I think these two issues are overlapping, at least in their root cause and
> solution. Added a suggestion to issue 3968 on how to fix them.
>
>
>
> Lieven
>
>
Received on 2012-05-21 20:58:44 CEST