Message-ID: <4E9438CD.20100@gmail.com>
Date: Tue, 11 Oct 2011 14:38:37 +0200
From: Yeb Havinga <yebhavinga@gmail.com>
User-Agent: Mozilla/5.0 (Windows NT 5.1;
	rv:7.0.1) Gecko/20110929 Thunderbird/7.0.1
MIME-Version: 1.0
To: Royce Ausburn <royce.ml@inomial.com>
CC: Pg Hackers <pgsql-hackers@postgresql.org>
Subject: Re: [REVIEW] Patch for cursor calling with named parameters
References: <FCEADDEF-8563-4965-8D4A-E37F2328C42B@inomial.com>
	<4E8ED2A7.2040800@gmail.com> <4E8F1315.70109@gmail.com>
	<29A8127B-BB37-40DC-8FB6-833710DAB22C@inomial.com>
In-Reply-To: <29A8127B-BB37-40DC-8FB6-833710DAB22C@inomial.com>
Content-Type: text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding: 7bit

Hello Royce,

Thanks again for testing.

On 2011-10-11 13:55, Royce Ausburn wrote:

> Just one small thing: it'd be nice to have an example for cursor declaration with named parameters.  Your patch adds one for opening a cursor, but not for declaring one.

Declaration of cursors with named parameters is already part of 
PostgreSQL (so it is possible to use the parameter names in the cursor 
query instead of $1, $2, etc.) and it also already documented with an 
example, just a few lines above the open examples. See curs3 on 
http://developer.postgresql.org/pgdocs/postgres/plpgsql-cursors.html

> Other than that, I think the patch is good.  Everything works as advertised =)

Thanks!

-- 
Yeb Havinga
http://www.mgrid.net/
Mastering Medical Data