Peter Eisentraut <[EMAIL PROTECTED]> writes:
> Tom Lane wrote:
>> What I'm inclined to do with these is change pg_proc.h but not force
>> an initdb.  Does anyone want to argue for an initdb to force it to be
>> fixed in 8.0?  We've lived with the wrong labelings for some time now
>> without noticing, so it doesn't seem like a serious enough bug to
>> force a post-beta initdb ... to me anyway.

> I'd prefer if all users of 8.0 were guaranteed to have the same catalog.  

Well, there's something to be said for that viewpoint too.  Anyone else
feel the same?

If we do go for an initdb, I'd like at the same time to do something
I had intended to do but forgotten, which is to yank the functions
and operators for basic arithmetic on type "char", and instead put in
(explicit) conversions between "char" and integer.  See for instance
http://archives.postgresql.org/pgsql-sql/2002-11/msg00116.php
http://archives.postgresql.org/pgsql-general/2004-08/msg01562.php
http://archives.postgresql.org/pgsql-general/2004-09/msg01209.php

Specifically I want to remove these operators:

 oid |        oid        | oprresult 
-----+-------------------+-----------
 635 | +("char","char")  | "char"
 636 | -("char","char")  | "char"
 637 | *("char","char")  | "char"
 638 | /("char","char")  | "char"

and their underlying functions:

 oid  |           oid            | prorettype |   prosrc    
------+--------------------------+------------+-------------
 1248 | charpl("char","char")    | "char"     | charpl
 1250 | charmi("char","char")    | "char"     | charmi
   77 | charmul("char","char")   | "char"     | charmul
   78 | chardiv("char","char")   | "char"     | chardiv

The following operators on "char" will remain:

 oid |        oid        | oprresult 
-----+-------------------+-----------
  92 | =("char","char")  | boolean
 630 | <>("char","char") | boolean
 631 | <("char","char")  | boolean
 632 | <=("char","char") | boolean
 633 | >("char","char")  | boolean
 634 | >=("char","char") | boolean

These are not as dangerous as the arithmetic operators, because in a
situation where the parser is having difficulty resolving types, it
will prefer the "text" comparison operators over these.  The reason
the "char" arithmetic operators are dangerous is that they are the only
ones of those names in the STRING type category.

                        regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 3: if posting/reading through Usenet, please send an appropriate
      subscribe-nomail command to [EMAIL PROTECTED] so that your
      message can get through to the mailing list cleanly

Reply via email to