| From: | David Rowley <david(dot)rowley(at)2ndquadrant(dot)com> |
|---|---|
| To: | Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com> |
| Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Simon Riggs <simon(at)2ndquadrant(dot)com>, Kouhei Kaigai <kaigai(at)ak(dot)jp(dot)nec(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Amit Kapila <amit(dot)kapila(at)enterprisedb(dot)com> |
| Subject: | Re: Combining Aggregates |
| Date: | 2016-01-21 04:52:47 |
| Message-ID: | CAKJS1f_8nbnFTgg_K-tFYJkaT-+6F1MvQxKaT2zabASioQanSQ@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 21 January 2016 at 15:53, Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com>
wrote:
> On Thu, Jan 21, 2016 at 1:33 PM, Haribabu Kommi
> <kommi(dot)haribabu(at)gmail(dot)com> wrote:
> >
> > Here I attached updated patch of parallel aggregate based on the latest
> > changes in master. Still it lack of cost comparison of normal aggregate
> to
> > parallel aggregate because of difficulty. This cost comparison is
> required
> > in parallel aggregate as this is having some regression when the number
> > of groups are less in the query plan.
> >
>
> Updated patch is attached after removing a warning in building group
> aggregate path.
Hi,
Thanks for updating the patch. I'd like to look at this with priority, but
can you post it on the Parallel Agg thread? that way anyone following there
can chime in over there rather than here. I've still got a bit of work to
do (in the not too distant future) on the serial/deserial part, so would be
better to keep this thread for discussion on that.
--
David Rowley http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Pavel Stehule | 2016-01-21 05:08:55 | Re: [PATCH] better systemd integration |
| Previous Message | Peter Geoghegan | 2016-01-21 04:46:56 | Re: Releasing in September |