subversion-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Kamesh Jayachandran <kam...@collab.net>
Subject [PATCH] Make svn clients indicate their operation name to backend(right now only to DAV) Version 2
Date Tue, 12 Jan 2010 13:19:10 GMT
Hi All,

Last week I posted the patch to implement 'svn client' to identify the 
svn operation they are about to do with a given ra_session.

Following thread has the detailed discussion.

http://mail-archives.apache.org/mod_mbox/subversion-dev/201001.mbox/%3C4B448959.1070400@collab.net%3E


Below is the summary:

Concern/Suggestion 1:
Michael Pilato and Philip Martin were suggesting to tweak 
libsvn_ra_(serf|neon) to detect the operation rather than making a 
change across all layers from the simplicity point of view.

My answer to 1:
I feel it would be too hackish to tweak one general API inside these 
modules to flag 'commit or write' operation to the server when the 
solution I propose handles this in a transparent way.

Concern/Suggestion 2:
There was a request to change the 'high_level_svn_operation' name at a 
later stage after opening the ra_session.

My answer to 2:
I have introduced a new API 'svn_ra_change_operation_name' to handle this.

Concern/Suggestion 3:
There was a request to change the header name from SVN-ACTION to 
something else as it will be a confusing with svn operational log's 
SVN-ACTION.

My answer to 3:
I have changed that request header to SVN-OPERATION.

I am attaching my new patch if there are no objections will commit it in 
Jan 14 my time.

With regards
Kamesh Jayachandran

Mime
View raw message