On Mon, Mar 14, 2016 at 8:44 AM, David Rowley <david.row...@2ndquadrant.com> wrote: > On 12 March 2016 at 16:31, David Rowley <david.row...@2ndquadrant.com> wrote: >> I've attached an updated patch which is based on commit 7087166, >> things are really changing fast in the grouping path area at the >> moment, but hopefully the dust is starting to settle now. > > The attached patch fixes a harmless compiler warning about a possible > uninitialised variable.
The setrefs.c fix for updating the finalize-aggregate target list is nice. I tested all the float aggregates and are working fine. Overall the patch is fine. I will do some test and provide the update later. Regards, Hari Babu Fujitsu Australia -- Sent via pgsql-hackers mailing list (firstname.lastname@example.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers