From: | Stephen Frost <sfrost(at)snowman(dot)net> |
---|---|
To: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | REVIEW: WIP: plpgsql - foreach in |
Date: | 2011-01-19 23:04:31 |
Message-ID: | 20110119230431.GQ4933@tamriel.snowman.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Greetings,
* Pavel Stehule (pavel(dot)stehule(at)gmail(dot)com) wrote:
> attached patch contains a implementation of iteration over a array:
I've gone through this patch and, in general, it looks pretty reasonable
to me. There's a number of places where I think additional comments
would be good and maybe some variable name improvments. Also, my
changes should be reviewed to make sure they make sense.
Attached is a patch against master which includes my changes, and a
patch against Pavel's patch, so he can more easily see my changes and
include them if he'd like.
I'm going to mark this returned to author with feedback.
commit 30295015739930e68c33b29da4f7ef535bc293ea
Author: Stephen Frost <sfrost(at)snowman(dot)net>
Date: Wed Jan 19 17:58:24 2011 -0500
Clean up foreach-in-array PL/PgSQL code/comments
Minor clean-up of the PL/PgSQL foreach-in-array patch, includes
some white-space cleanup, grammar fixes, additional errhint where
it makes sense, etc.
Also added a number of 'XXX' comments asking for clarification
and additional comments on what's happening in the code.
commit f1a02fe3a8fa84217dae32d5ba74e9764c77431c
Author: Stephen Frost <sfrost(at)snowman(dot)net>
Date: Wed Jan 19 15:11:53 2011 -0500
PL/PgSQL - Add interate-over-array support
This patch adds support for iterating over an array in PL/PgSQL.
Patch Author: Pavel Stehule
Thanks,
Stephen
Attachment | Content-Type | Size |
---|---|---|
test_plpgsql_array.patch | text/x-diff | 24.4 KB |
test_plpgsql_array_cleanup.patch | text/x-diff | 13.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Tomas Vondra | 2011-01-19 23:32:37 | Re: estimating # of distinct values |
Previous Message | Andreas Karlsson | 2011-01-19 22:47:11 | Re: psql: Add \dL to show languages |