Re: svn commit: rev 2555 - trunk/subversion/include trunk/subversion/libsvn_wc trunk/subversion/libsvn_client trunk/subversion/libsvn_ra_dav

Greg Stein <gstein@lyra.org> writes:

> I'd say: fix the logging by adding wcprops into the mix and making it more
> transacted. Then rip out this new RA code.

Here's some more argument against doing the above.

Right now, the RA layers have two property functions to play with, the
prop-get and prop-set functions. If we make the prop-set loggy, you
have two choices:

  1. Run the log immediately -- well, this won't fix the problem Karl
      set out to take care of. I mean, we will still have all the
      vsn-rsrc-urls written out before any of the entries ever get
      bumped.

  2. Don't run the log at all -- well now you have this *extremely*
      broken API in which an RA layer can set a property `foo' to
      `bar' and then immediately try to read it...and it's not there,
      or still has an out-of-date value! That's just silly.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Jul 17 18:35:59 2002