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 <[email protected]> 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: [email protected] [mailto:[email protected]] On Behalf > Of Lieven Govaerts > Sent: maandag 21 mei 2012 19:28 > To: Johan Corveleyn; [email protected] > Cc: Greg Stein > > > Subject: Re: [DISCUSS] delete ra_neon > > > > > > On Mon, May 21, 2012 at 1:47 PM, Johan Corveleyn <[email protected]> wrote: > > On Sat, May 19, 2012 at 9:58 PM, Greg Stein <[email protected]> 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 > >
