On Mon, Mar 14, 2016 at 8:44 AM, David Rowley
<david.rowley@2ndquadrant.com> wrote:
> On 12 March 2016 at 16:31, David Rowley <david.rowley@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