| From: | Andres Freund <andres(at)anarazel(dot)de> |
|---|---|
| To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
| Cc: | David Rowley <david(dot)rowley(at)2ndquadrant(dot)com>,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-05-20 15:23:51 |
| Message-ID: | 50AC73BA-F4BE-4F06-A5E6-C8CD7A28B979@anarazel.de |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
On May 20, 2019 6:23:46 AM PDT, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>On Sun, May 19, 2019 at 2:36 PM Andres Freund <andres(at)anarazel(dot)de>
>wrote:
>> Not sure I understand the distinction you're trying to make with the
>> variable renaming. The combine function is also a transition
>function,
>> no?
>
>Not in my mental model. It's true that a combine function is used in
>a similar manner to a transition function, but they are not the same
>thing.
Well, the context here is precisely that. We're still calling functions that have trans* in the name, we pass them transfn style named parameters. If you read my suggestion, it essentially is running go *further* than David's renaming?
Andres
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Konstantin Knizhnik | 2019-05-20 15:28:43 | Re: PROXY protocol support |
| Previous Message | Dean Rasheed | 2019-05-20 15:09:24 | Re: Multivariate MCV stats can leak data to unprivileged users |