[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1656694 - in /subversion/trunk/subversion: include/private/svn_ra_private.h include/svn_ra.h libsvn_client/diff.c libsvn_ra/ra_loader.c

From: Branko Čibej <brane_at_wandisco.com>
Date: Tue, 03 Feb 2015 15:35:42 +0100

On 03.02.2015 13:28, Julian Foad wrote:
> Branko Čibej wrote:
>> On 03.02.2015 12:49, rhuijben_at_apache.org wrote:
>>> URL: http://svn.apache.org/r1656694
>>> Log:
>>> Rename svn_ra_dup_session() (public api) to svn_ra__dup_session (private
>> api).
>>
>> +1000
> Presumably you both have in mind some reason why this change is beneficial. (I saw you discussing on IRC about it being "unsafe" or some such.) Could you please hint at the reason for this, in the log message?

There are severe constraints on where and how one may use the duplicate
session. Rather than explain them at length in docs and deal with tons
of spurious bug reports, I think it's better to keep this for private
use within libsvn_client; *especially* if we manage to merge the
reuse-ra-session branch to trunk, where use of duplicate sessions that
may be based on something that's managed by libsvn_client's session
cache is far too error-prone to be useful.

The RA session cache will, in most cases, make session duplication less
useful anyway.

-- Brane
Received on 2015-02-03 15:36:35 CET

This is an archived mail posted to the Subversion Dev mailing list.