Received: from magus.postgresql.org (magus.postgresql.org [87.238.57.229])
	by mail.postgresql.org (Postfix) with ESMTP id D0B9619CDE12
	for <pgsql-hackers@postgresql.org>;
	Mon,  5 Dec 2011 13:58:25 -0400 (AST)
Received: from mailout-de.gmx.net ([213.165.64.23])
	by magus.postgresql.org with smtp (Exim 4.72)
	(envelope-from <peter_e@gmx.net>) id 1RXco4-0004PZ-7X
	for pgsql-hackers@postgresql.org; Mon, 05 Dec 2011 17:58:25 +0000
Received: (qmail invoked by alias); 05 Dec 2011 17:58:10 -0000
Received: from a88-115-218-165.elisa-laajakaista.fi (EHLO [10.0.0.102])
	[88.115.218.165]
	by mail.gmx.net (mp063) with SMTP; 05 Dec 2011 18:58:10 +0100
X-Authenticated: #495269
X-Provags-ID: V01U2FsdGVkX19AhLcKOUFAwGLTQFhtkjOoeitHHiAY4NVVE4w+uY
	SLdoIbshTm+xmr
Message-ID: <1323107887.10992.10.camel@vanquo.pezone.net>
Subject: Re: plpython SPI cursors
From: Peter Eisentraut <peter_e@gmx.net>
To: Jan =?UTF-8?Q?Urba=C5=84ski?= <wulczer@wulczer.org>
Cc: Steve Singer <ssinger_pg@sympatico.ca>, Postgres - Hackers
	<pgsql-hackers@postgresql.org>
Date: Mon, 05 Dec 2011 19:58:07 +0200
In-Reply-To: <4ECD426F.5060702@wulczer.org>
References: <4E9A1738.6090108@wulczer.org>
	<BLU0-SMTP4530C1907601DFB9AEAE778ECA0@phx.gbl>
	<4ECD426F.5060702@wulczer.org>
Content-Type: text/plain; charset="UTF-8"
X-Mailer: Evolution 3.0.3-2 
Content-Transfer-Encoding: 8bit
Mime-Version: 1.0
X-Y-GMX-Trusted: 0
X-Pg-Spam-Score: -3.1 (---)
X-Archive-Number: 201112/201
X-Sequence-Number: 199453

On ons, 2011-11-23 at 19:58 +0100, Jan Urbański wrote:
> On 20/11/11 19:14, Steve Singer wrote:
> > On 11-10-15 07:28 PM, Jan Urbański wrote:
> >> Hi,
> >>
> >> attached is a patch implementing the usage of SPI cursors in PL/Python.
> >> Currently when trying to process a large table in PL/Python you have
> >> slurp it all into memory (that's what plpy.execute does).
> >>
> >> J
> >
> > I found a few bugs (see my testing section below) that will need fixing
> > + a few questions about the code
> 
> Responding now to all questions and attaching a revised patch based on 
> your comments.

Committed

Please refresh the other patch.