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

Re: undo records (was: svn commit: rev 7096 - in trunk/subversion: libsvn_fs tests/libsvn_fs)

From: C.Michael Pilato <cmpilato_at_collab.net>
Date: 2003-09-24 20:25:34 CEST

Greg Stein <gstein@lyra.org> writes:

> > location) basically, he didn't want a node-revision's in-memory
> > representation to outlive its trail because other processes might
> > change the stored version of that node-revision -- so in order to make
> > changes to a node-revision, you had to read the node-revision from the
> > database and verify that (instead of just pulling up an old cached
> > copy of the thing), and then write back the modified node-revision in
> > the same Berkeley transaction.
>
> Euh... is there any good reason why it is cached? And now that we have a
> scratchpool, couldn't we just add a cleanup function there? Or maybe just
> allocate the dag node and the cached node revision *in* that scratchpool.

We used to cache it back when it held all the node's contents. It's
possible that this no longer needs to happen. Personally, I'd like
the to see the 'node_revision' member removed from the dag_node_t --
keeping it there is just begging for trouble.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Sep 24 20:29:03 2003

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.