[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: Sander Striker <striker_at_apache.org>
Date: 2004-02-25 01:21:46 CET

On Tue, 2004-02-24 at 20:16, kfogel@collab.net wrote:
> 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
> database.
> 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.)
> Thoughts?

Yes, turn it around: --disable-txnless-reporter if you don't want it.
People tracking trunk are doing so for a reason. Don't you love
ginea pigs? ;) :)


To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Feb 25 01:21:22 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.