Re: Various breakages in new contrib/isn module

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: kronuz(at)hotmail(dot)com
Cc: pgsql-hackers(at)postgreSQL(dot)org
Subject: Re: Various breakages in new contrib/isn module
Date: 2006-11-23 19:34:02
Message-ID: 25313.1164310442@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

I wrote:
> * There are a whole bunch of "shell" operators created; try
> select oid::regoperator from pg_operator where oprcode = 0;
> after loading isn.

Actually, the problem with this module is not that it's got too few
operators, but that it's got too many. Trying to have the cross-product
of all possible comparison operators for eight nominally different types
is not very sane.

If we were to make the casts to ean13 be IMPLICIT instead of assignment,
then we could have one set of comparison operators and one opclass for
ean13, and let the restricted types rely on those implicitly. Is there
a good reason to disallow the implicit promotion?

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Stephen Harris 2006-11-23 19:38:27 Re: Shutting down a warm standby database in 8.2beta3
Previous Message Tom Lane 2006-11-23 18:31:39 Re: 8.2 open items list