On 18 September 2015 at 01:50, Stefan Fuhrmann <stefan.fuhrm...@wandisco.com> wrote: > On Fri, Sep 18, 2015 at 12:08 AM, Ivan Zhakov <i...@visualsvn.com> wrote: >> >> On 17 September 2015 at 21:53, Philip Martin <philip.mar...@wandisco.com> >> wrote: >> > Ivan Zhakov <i...@visualsvn.com> writes: >> > [...]
>> > Detect the error from a broken RA >> > session and create another? Track the time when the session was last >> > used? Something else? >> > >> Current implementation tracks last time when session was used and do >> not reuse RA sessions that was inactive for 5 minutes. > > > My 2 cents: I wonder whether 5 minutes are a good default > or whether 0.5 .. 1 minute wouldn't be a better one. A lower > value allows for faster recovery from intermittent failures. > > The typical use-cases for this feature are c/o with many > svn:externals and interactive GUIs like TSVN's repo browser. > Both should be fine with timeouts of about 1 minute. But this > is just a small point that sprang to my attention and I don't feel > like we need to have a discussion on this topic right now. > Fully agree. I changed expiry timeout to 1 minute in r1703803. Actually I thought that RA expiry timeout was 5 *seconds* :) -- Ivan Zhakov
