Re: PL/Python array support

From: Joshua Tolley <eggyknap(at)gmail(dot)com>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: Teodor Sigaev <teodor(at)sigaev(dot)ru>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: PL/Python array support
Date: 2009-12-02 12:11:56
Message-ID: 20091202121156.GC20931@eddie
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Nov 20, 2009 at 12:00:24AM +0200, Peter Eisentraut wrote:
> On fre, 2009-11-13 at 18:46 +0300, Teodor Sigaev wrote:
> > CREATE OR REPLACE FUNCTION incr(stuff int[]) RETURNS int[] AS $$
> > for x in stuff:
> > yield x+1
> > $$
> > LANGUAGE 'plpythonu';
> >
> > # select incr(ARRAY[1,2,3]);
> > ERROR: invalid memory alloc request size 18446744073709551608
> > CONTEXT: while creating return value
> > PL/Python function "incr"
>
> Fixed with additional error check and regression test. (The problem
> could be more simply demonstrated by returning any non-sequence from the
> function.) Thanks for catching it.

My last email claimed that the regression test needed some additional changes
to its expected output, and further claimed that it had the regression test's
diff attached. As was helpfully pointed out off-list, it actually wasn't
attached. Trying again..

-- Josh

Attachment Content-Type Size
regression.diffs text/plain 723 bytes

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Greg Stark 2009-12-02 12:42:16 Re: Cost of sort/order by not estimated by the query planner
Previous Message Dimitri Fontaine 2009-12-02 11:26:39 Re: Page-level version upgrade