Re: [PATCH v2] HTTPv2 allow client to control transaction name in protocol

On 03/04/2011 07:29 AM, Philip Martin wrote:
> "C. Michael Pilato" <cmpilato_at_collab.net> writes:
>
>> Just a thought: Have you considered expanding the scope of the private
>> resource space rather than using the magic prefix hack? You could add
>> ".../!svn/vtxn/UUID" and ".../!svn/vtxr/UUID/..." to be alternate ways to
>> address transactions and transaction roots (the "v" there being a shortcut
>> for "virtual"). This is *effectively* the same approach as yours -- there's
>> a different prefix here. But the prefix is a clearly defined piece of the
>> protocol, not just some magic bit buried in mod_dav_svn's codebase.
>
> This looks promising, it's quite straightforward to implement. New
> log/patch follows:

Oh, that's *so* much more satisfying. Most of this kind of work is
copy-n-paste-n-tweak, I know, but I gave it a once-over anyway. Looks good
to me!

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand