| From: | David Rowley <david(dot)rowley(at)2ndquadrant(dot)com> |
|---|---|
| To: | Andres Freund <andres(at)anarazel(dot)de> |
| Cc: | Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>, Rajkumar Raghuwanshi <rajkumar(dot)raghuwanshi(at)enterprisedb(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Statistical aggregate functions are not working with PARTIAL aggregation |
| Date: | 2019-07-24 22:36:26 |
| Message-ID: | CAKJS1f_NnttW6tZBkFQ9Un3i8+iomE45qhAKth4yeBRU4o7eHA@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Thu, 25 Jul 2019 at 06:52, Andres Freund <andres(at)anarazel(dot)de> wrote:
> Now that master is open for development, and you have a commit bit, are
> you planning to go forward with this on your own?
I plan to, but it's not a high priority at the moment. I'd like to do
much more in nodeAgg.c, TBH. It would be good to remove some code from
nodeAgg.c and put it in the planner.
I'd like to see:
1) Planner doing the Aggref merging for aggregates with the same transfn etc.
2) Planner trying to give nodeAgg.c a sorted path to work with on
DISTINCT / ORDER BY aggs
3) Planner providing nodeAgg.c with the order that the aggregates
should be evaluated in order to minimise sorting for DISTINCT / ORDER
BY aggs.
I'd take all those up on a separate thread though.
If you're in a rush to see the cleanup proposed a few months ago then
please feel free to take it up. It might be a while before I can get a
chance to look at it again.
--
David Rowley http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andres Freund | 2019-07-24 23:24:39 | ON CONFLICT (and manual row locks) cause xmax of updated tuple to unnecessarily be set |
| Previous Message | Fabien COELHO | 2019-07-24 22:26:34 | Re: pgbench - allow to create partitioned tables |