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: | Raw Message | Whole Thread | 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 |