| From: | Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com> |
|---|---|
| To: | David Rowley <david(dot)rowley(at)2ndquadrant(dot)com> |
| Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Paul Ramsey <pramsey(at)cleverelephant(dot)ca>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Parallel Aggregate |
| Date: | 2016-03-14 00:33:19 |
| Message-ID: | CAJrrPGeKKPDpioeJv6CAgmRta7rDLBt_y=znWQcS5B=8b_zetA@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Mon, Mar 14, 2016 at 8:44 AM, David Rowley
<david(dot)rowley(at)2ndquadrant(dot)com> wrote:
> On 12 March 2016 at 16:31, David Rowley <david(dot)rowley(at)2ndquadrant(dot)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
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tomas Vondra | 2016-03-14 01:00:03 | Re: Re: PATCH: Split stats file per database WAS: autovacuum stress-testing our system |
| Previous Message | Thomas Munro | 2016-03-14 00:20:50 | Re: WIP: Detecting SSI conflicts before reporting constraint violations |