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 |
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 |