Re: Second thoughts on CheckIndexCompatible() vs. operator families - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Second thoughts on CheckIndexCompatible() vs. operator families
Date
Msg-id 5181.1327791941@sss.pgh.pa.us
Whole thread Raw
In response to Re: Second thoughts on CheckIndexCompatible() vs. operator families  (Noah Misch <noah@leadboat.com>)
List pgsql-hackers
Noah Misch <noah@leadboat.com> writes:
> On Thu, Jan 26, 2012 at 08:23:34AM -0500, Robert Haas wrote:
>> I'm just going to remove the test.  This is not very future-proof and

> [ objections ]

FWIW, I concur with Robert's choice here.  This test method is ugly and
fragile, and I'm not even thinking about the question of whether an
installation might have GUC settings that affect it.  My objection is
that you're assuming that nowhere else, anywhere in the large amount of
code executed by the queries under test, will anyone ever wish to insert
another elog(DEBUG) message.

> I used the same strategy in another ALTER TABLE patch this CF:
> http://archives.postgresql.org/message-id/20120126033956.GC15670@tornado.leadboat.com

That's going to need to be removed before commit too, then.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Jan Urbański
Date:
Subject: unfriendly error when accessing NEW in statement-level trigger
Next
From: Tom Lane
Date:
Subject: Re: unfriendly error when accessing NEW in statement-level trigger