| From: | Marti Raudsepp <marti(at)juffo(dot)org> |
|---|---|
| To: | Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)oss(dot)ntt(dot)co(dot)jp> |
| Cc: | pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: [REVIEW] Generate column names for subquery expressions |
| Date: | 2011-09-18 23:28:14 |
| Message-ID: | CABRT9RCm=rDPogJ8+v0maHPEisUuDf2=02K0=sGCq2iwWjtqeg@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, Sep 14, 2011 at 05:26, Kyotaro HORIGUCHI
<horiguchi(dot)kyotaro(at)oss(dot)ntt(dot)co(dot)jp> wrote:
> This is a review for the patch `Generate column names for
> subquery expressions'
> (https://commitfest.postgresql.org/action/patch_view?id=632)
Thanks for the review. :)
PS: When you send a review, you should add the author's email to the
"To:" line to make sure they see it. I noticed your email only today
because it was in a new thread and not addressed to me directly.
> I think this patch needs no documentation, but it is needed to
> edit the changed behaviors quoted in document. Maybe only one
> change as far as I have seen.
>
> http://www.postgresql.org/docs/9.0/static/sql-expressions.html
>> SELECT ARRAY(SELECT oid FROM pg_proc WHERE proname LIKE 'bytea%');
>> ?column?
>> -------------------------------------------------------------
>> {2011,1954,1948,1952,1951,1244,1950,2005,1949,1953,2006,31}
Good catch, a new patch is attached. Apparently the results of this
query have also changed in recent versions, but I didn't touch that.
Regards,
Marti
| Attachment | Content-Type | Size |
|---|---|---|
| Generate-column-names-for-subquery-expressions-v3.patch | text/x-patch | 4.5 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2011-09-19 00:11:26 | Re: unite recovery.conf and postgresql.conf |
| Previous Message | Marti Raudsepp | 2011-09-18 22:25:46 | Re: [WIP] Caching constant stable expressions per execution |