Re: sinval contention reduction

From: Simon Riggs <simon(at)2ndquadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: sinval contention reduction
Date: 2008-01-28 09:25:51
Message-ID: 1201512351.4257.625.camel@ebony.site
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

On Sat, 2008-01-26 at 14:27 -0500, Tom Lane wrote:
> Simon Riggs <simon(at)2ndquadrant(dot)com> writes:
> > On Fri, 2008-01-25 at 19:02 -0500, Tom Lane wrote:
> >> This seems large, complex, and untested (I note in particular a
> >> guaranteed-to-fail Assert).
>
> > Yes, its for discussion. How would you describe such a patch in the
> > future? I want to be able to differentiate patch status.
>
> "Completely untested" might be an appropriate description ...

That wouldn't be true, because it passes make check. If it were true,
I'd have said it.

Your responses are inappropriate to a patch clearly marked "for
discussion", especially when you privately suggested this topic for me
to look at and you also know exactly which system I was going to run a
performance test on once I had the patch agreed.

--
Simon Riggs
2ndQuadrant http://www.2ndQuadrant.com

In response to

Browse pgsql-patches by date

  From Date Subject
Next Message Zeugswetter Andreas ADI SD 2008-01-28 11:25:45 Re: [PATCHES] Proposed patch: synchronized_scanning GUC variable
Previous Message Dean Rasheed 2008-01-28 09:21:50 Auto-explain patch