Re: [INTERFACES] DBD::Pg installation seems to fail with 7.1 libs - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [INTERFACES] DBD::Pg installation seems to fail with 7.1 libs
Date
Msg-id 8277.973288548@sss.pgh.pa.us
Whole thread Raw
List pgsql-hackers
Frank Joerdens <frank@joerdens.de> writes:
> [ DBD::Pg fails against current sources with ]
> DBD::Pg::db table_info failed: ERROR:  Unable to identify an ordering
> operator '<' for type 'unknown'

Hmm.  It looks like this is caused by my reimplementation of UNION to
use CASE-style datatype resolution.  The DBD code is fetching table
data with a UNION that includes a couple of untyped literal constants.
Boiled down:

regression=# select 'foo' union select 'bar';
ERROR:  Unable to identify an ordering operator '<' for type 'unknown'       Use an explicit ordering operator or
modifythe query
 

This is bad, since the same query used to work pre-7.1.

The reason it worked is that the old UNION code had an ugly hack to
force such constants to be treated as type text.  There is no such hack
in the datatype unification code right now.

Thomas, you've been muttering about altering the type resolution rules
so that "unknown" will be treated as "text" when all else fails.  Are
you planning to commit such a thing for 7.1?  If not, I'll probably have
to hack up parse_coerce.c's select_common_type(), along the lines of
             }         }     }
+    /* if all inputs are UNKNOWN, treat as text */
+     if (ptype == UNKNOWNOID)
+        ptype = TEXTOID;     return ptype; }


This might be an appropriate change anyway.  Comments?
        regards, tom lane


pgsql-hackers by date:

Previous
From: lsearchw@altavista.net
Date:
Subject: Re: me too
Next
From: Vince Vielhaber
Date:
Subject: Re: me too