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

Re: [PATCH] Change svn_fs_root_t to include pointer to related txn

From: Michael Pilato <cmpilato_at_collab.net>
Date: 2005-12-14 21:36:15 CET

On Wed, 2005-12-14 at 11:19 -0800, Jim Blandy wrote:
> On 12/14/05, Greg Hudson <ghudson@mit.edu> wrote:
> > Instead of creating or opening a txn, you'd create or open a txn root.
> > There's no such thing as a "revision" object in the svn_fs interface, so
> > I don't see the need for a "transaction" object either.
>
> So, svn_fs_commit_txn and svn_fs_abort_txn would be applied to the
> transaction root, instead of the transaction object?

Wouldn't they be applied to the transaction name instead of the object?
Those are not, and should remain not, operations a "root".

FWIW, this change (losing svn_fs_txn_t altogether) is one I intended to
make pre-1.0, even, but never got around to it. I say that only to say,
+1 on the concept.

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

Received on Wed Dec 14 21:38:50 2005

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.