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

Re: Transactionless reporter again

From: <kfogel_at_collab.net>
Date: 2004-02-24 20:16:24 CET

Greg Hudson <ghudson@MIT.EDU> writes:
> I'd like to commit my transactionless reporter code, which I described
> in
> http://subversion.tigris.org/servlets/BrowseList?list=dev&by=thread&from=143289
> and put in the issue tracker in
> http://subversion.tigris.org/issues/show_bug.cgi?id=1679
> a couple of months ago.
> There are no API changes involved; it's just a performance
> improvement, and there should be no problem to have it in svn 1.1.
> Object now or don't.

You're talking about replacing our current reporter.c with the new
code, so that the transactionless reporter becomes the new default?

I don't object to this; I already reviewed the patch back when you
first posted it. Spooling the file also feels funny to me, but if
pressed I have to admit that it's not worse than spooling into the

However, it might be wise to add this as a conditional part of the
build first, e.g.,

   $ ./configure --enable-txnless-reporter

...or something, so we can get some more testing first. After a few
weeks, we could switch over completely.

(I would add this option to my builds, so that I'm always running the
new code. Suspect you can get similar commitments from others.)



To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Feb 24 21:19:53 2004

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.