Thread: DBD::Pg installation seems to fail with 7.1 libs

DBD::Pg installation seems to fail with 7.1 libs

From
Frank Joerdens
Date:
Here´s the error:

------------------------- snip -------------------------
DBD::Pg::db table_info failed: ERROR:  Unable to identify an ordering
operator '<' for type 'unknown'       Use an explicit ordering operator or modify the query
Can't call method "fetchrow_array" on an undefined value at test.pl line
103. 
------------------------- snap -------------------------

and this is the part of the test.pl script referred to by the error
message:


------------------------- snip -------------------------######################### create table    85    86
$dbh->do("CREATETABLE builtin (    87   bool_      bool,    88   char_      char,    89   char12_    char(12),    90
char16_   char(16),    91   varchar12_ varchar(12),    92   text_      text,    93   date_      date,    94   int4_
int4,    95   int4a_     int4[],    96   float8_    float8,    97   point_     point,    98   lseg_      lseg,    99
box_      box   100   )");   101   102 $sth = $dbh->table_info;   103 my @infos = $sth->fetchrow_array; 
 
------------------------- snap -------------------------

Now, since it works with 6.4, and the developer of DBD::Pg says that
he´s tested this with 6.5, my hunch is that it has to do with some
change or other that was made in 7.1.

Has anyone tried to install or use DBI and DBD::Pg with the current
snapshot? The snapshot I used is from October 10.

I haven´t tested this with the current version, i.e. release 7, but will
probably do that if I get round to it.

Cheers Frank

-- 
frank joerdens               

joerdens new media
urbanstr. 116
10967 berlin
germany

e: frank@joerdens.de
t: +49 (0)30 69597650
f: +49 (0)30 7864046 
h: http://www.joerdens.de

pgp public key: http://www.joerdens.de/pgp/frank_joerdens.asc


Re: DBD::Pg installation seems to fail with 7.1 libs

From
Tom Lane
Date:
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


Re: DBD::Pg installation seems to fail with 7.1 libs

From
Thomas Lockhart
Date:
> 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

I'm *finally* getting several patches together, to do the following
things:

o Fix the type resolution for unknown function arguments to fall back to
"text" or a string type, if available. Previously discussed.

o Implement an AT TIME ZONE clause, per SQL9x. Will handle an INTERVAL
argument, per standard, and also accept a string containing a time zone
spec, per existing PostgreSQL extension. Previously discussed.

o Fix timestamp/interval math across daylight savings time boundaries.
Previously discussed.

o Allow interpretation of "hh:mm:ss" as INTERVAL input. I can't remember
if I've mentioned this one before.

o Fix output of INTERVAL when sign of year/month is different than sign
of hour/min/sec. This is accompanied by changes in the "ISO" form of
output to more closely resemble a "hh:mm:ss" format. I just noticed the
problem today, so have not discussed it on list yet.

o Add some JOIN regression tests. More should be and will be done, but I
don't want to keep holding back patches on this. Per Tom Lane's request.


The only one with some potential for user trouble is the INTERVAL format
change. The old code was wrong, but the format itself has been changed
to be a little more concise.
                      - Thomas


Re: [HACKERS] Re: DBD::Pg installation seems to fail with 7.1 libs

From
Frank Jördens
Date:
Thomas Lockhart wrote:
> 
> > 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
> 
> I'm *finally* getting several patches together, to do the following
> things:
> 
> o Fix the type resolution for unknown function arguments to fall back to
> "text" or a string type, if available. Previously discussed.

Can this be expected to be found in the 7.1 snapshot release anytime
soon? Otherwise I would have to continue developing my app on 7.0.x,
which is not so good because eventually I will be requiring TOAST and
I'd like to run into 7.1 related trouble - should there be such a thing
;) - as early as possible.

Many Thanks, Frank


Re: [HACKERS] Re: DBD::Pg installation seems to fail with 7.1 libs

From
Frank Joerdens
Date:
Frank Jördens wrote:
> 
> Thomas Lockhart wrote:
> >
> > > 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
> >
> > I'm *finally* getting several patches together, to do the following
> > things:
> >
> > o Fix the type resolution for unknown function arguments to fall back to
> > "text" or a string type, if available. Previously discussed.
> 
> Can this be expected to be found in the 7.1 snapshot release anytime
> soon? Otherwise I would have to continue developing my app on 7.0.x,

works a treat with the changes committed last thursday :)

- Frank


Re: [HACKERS] Re: DBD::Pg installation seems to fail with 7.1 libs

From
Thomas Lockhart
Date:
> > o Fix the type resolution for unknown function arguments to fall back to
> > "text" or a string type, if available. Previously discussed.
> Can this be expected to be found in the 7.1 snapshot release anytime
> soon?

It is already there, and even has had a round of bug fixes already :)

I expect that it will work for you with no problems.
                 - Thomas