On 18 September 2015 at 01:50, Stefan Fuhrmann
<stefan.fuhrmann_at_wandisco.com> wrote:
> On Fri, Sep 18, 2015 at 12:08 AM, Ivan Zhakov <ivan_at_visualsvn.com> wrote:
>>
>> On 17 September 2015 at 21:53, Philip Martin <philip.martin_at_wandisco.com>
>> wrote:
>> > Ivan Zhakov <ivan_at_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
Received on 2015-09-18 13:21:56 CEST