pg.ddx.io  pgsql-interfaces@postgresql.org mailing list archive  
help / color / mirror / Atom feed
Type implemented in plpythonu crashes backend
3+ messages / 3 participants
[nested] [flat]

* Type implemented in plpythonu crashes backend
@ 2004-08-11 07:48 pgsql@groks.org
  2004-08-15 19:02 ` Re: Type implemented in plpythonu crashes backend Tom Lane <tgl@sss.pgh.pa.us>
  0 siblings, 1 reply; 3+ messages in thread

From: pgsql@groks.org @ 2004-08-11 07:48 UTC (permalink / raw)
  To: pgsql-interfaces



I'd like to implement new postgres types in a language other than C for
prototyping speed.  plpgsql won't let me create a function which takes a
cstring, and neither will pltcl.  Happily, plpythonu will, and it seems
to work fine.

However, the code below causes the backend to crash...

I'm guessing that plpython is trying to format the return value using
the appropriate registered function, which just happens to be itself.


My goal is to prototype new types, operators and gist indexes without
getting bogged down in with the guts of posgresql in C, so if it's
unreasonable to expect the below code to work, do you have any other
suggestions?


Thanks.




create or replace function pycomplex_in(cstring)
returns opaque as '

import struct
import re

p = re.compile(" *\\( *([0-9]+(\\.[0-9]+)?) *, *([0-9]+(\\.[0-9]+)?)*\\)*")
m = p.match(args[0])

if m:
    (x, y) = m.group(1, 3)
else:
    (x, y) = (0, 0)
    #plpy.error("incorrect format for pycomplex: %s" % args[0])

return struct.pack("dd", float(x), float(y))

' language plpythonu immutable strict;



create or replace function pycomplex_out(opaque)
returns cstring as '

import struct

(x, y) = struct.unpack("dd", args[0])
return "(%f, %f)", (x, y)

' language plpythonu immutable strict;


create type pycomplex (
   internallength = 16,
   input          = pycomplex_in,
   output         = pycomplex_out,
   alignment      = double
);


select pycomplex_out(pycomplex_in('(1,2)'));







^ permalink  raw  reply  [nested|flat] 3+ messages in thread

* Re: Type implemented in plpythonu crashes backend
  2004-08-11 07:48 Type implemented in plpythonu crashes backend pgsql@groks.org
@ 2004-08-15 19:02 ` Tom Lane <tgl@sss.pgh.pa.us>
  2004-08-17 02:45   ` Re: Type implemented in plpythonu crashes backend Bruce Momjian <pgman@candle.pha.pa.us>
  0 siblings, 1 reply; 3+ messages in thread

From: Tom Lane @ 2004-08-15 19:02 UTC (permalink / raw)
  To: pgsql@groks.org; +Cc: pgsql-interfaces

pgsql@groks.org writes:
> I'd like to implement new postgres types in a language other than C for
> prototyping speed.  plpgsql won't let me create a function which takes a
> cstring, and neither will pltcl.  Happily, plpythonu will, and it seems
> to work fine.

I'd say this is an oversight in the plpython code --- it should be
rejecting pseudotype arguments, since as you've discovered it cannot
really cope.  It should *certainly* reject "opaque" ... might be okay to
allow cstring but I'm not sure.

			regards, tom lane



^ permalink  raw  reply  [nested|flat] 3+ messages in thread

* Re: Type implemented in plpythonu crashes backend
  2004-08-11 07:48 Type implemented in plpythonu crashes backend pgsql@groks.org
  2004-08-15 19:02 ` Re: Type implemented in plpythonu crashes backend Tom Lane <tgl@sss.pgh.pa.us>
@ 2004-08-17 02:45   ` Bruce Momjian <pgman@candle.pha.pa.us>
  0 siblings, 0 replies; 3+ messages in thread

From: Bruce Momjian @ 2004-08-17 02:45 UTC (permalink / raw)
  To: Tom Lane <tgl@sss.pgh.pa.us>; +Cc: pgsql@groks.org; pgsql-interfaces


Added to open items list:

	* have plpython reject pseudotype arguments because it crashes

---------------------------------------------------------------------------

Tom Lane wrote:
> pgsql@groks.org writes:
> > I'd like to implement new postgres types in a language other than C for
> > prototyping speed.  plpgsql won't let me create a function which takes a
> > cstring, and neither will pltcl.  Happily, plpythonu will, and it seems
> > to work fine.
> 
> I'd say this is an oversight in the plpython code --- it should be
> rejecting pseudotype arguments, since as you've discovered it cannot
> really cope.  It should *certainly* reject "opaque" ... might be okay to
> allow cstring but I'm not sure.
> 
> 			regards, tom lane
> 
> ---------------------------(end of broadcast)---------------------------
> TIP 8: explain analyze is your friend
> 

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073



^ permalink  raw  reply  [nested|flat] 3+ messages in thread


end of thread, other threads:[~2004-08-17 02:45 UTC | newest]

Thread overview: 3+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2004-08-11 07:48 Type implemented in plpythonu crashes backend pgsql@groks.org
2004-08-15 19:02 ` Tom Lane <tgl@sss.pgh.pa.us>
2004-08-17 02:45   ` Bruce Momjian <pgman@candle.pha.pa.us>

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox