From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com> |
Cc: | PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: FILTER for aggregates [was Re: Department of Redundancy Department: makeNode(FuncCall) division] |
Date: | 2013-07-08 01:37:26 |
Message-ID: | 20130708013726.GG18753@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sat, Jul 06, 2013 at 11:49:21AM +0100, Dean Rasheed wrote:
> On 5 July 2013 18:23, David Fetter <david(at)fetter(dot)org> wrote:
> > Please find attached changes based on the above.
> >
>
> This looks good. The grammar changes are smaller and neater now on top
> of the makeFuncCall() patch.
>
> Overall I think this patch offers useful additional functionality, in
> compliance with the SQL spec, which should be handy to simplify
> complex grouping queries.
>
> There's a minor typo in syntax.sgml: "for each input row, each row
> matching same." --- fix attached.
That is actually correct grammar, but may not be the easiest to
translate.
> I think this is ready for committer.
Thanks :)
Cheers,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics
Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2013-07-08 03:14:34 | Re: postgresMain() function |
Previous Message | Noah Misch | 2013-07-08 01:28:34 | Re: Re: [Review] Add SPI_gettypmod() to return a field's typemod from a TupleDesc |