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? ;) :)
Sander
---------------------------------------------------------------------
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