From: | Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Florian Pflug <fgp(at)phlo(dot)org>, David Rowley <dgrowleyml(at)gmail(dot)com>, Kevin Grittner <kgrittn(at)ymail(dot)com>, Josh Berkus <josh(at)agliodbs(dot)com>, Greg Stark <stark(at)mit(dot)edu>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [PATCH] Negative Transition Aggregate Functions (WIP) |
Date: | 2014-04-10 19:34:56 |
Message-ID: | CAEZATCWOOK8zWW2zwT8GQXLfJZJXGy6caV2BPC5xO__i5RHPaQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 10 April 2014 19:54, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com> writes:
>> On 10 April 2014 19:04, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>>> What about names for the invertible-aggregate infrastructure?
>>> I'm tempted to prefix "inv" to all the existing names, but then
>>> "invsfunc" means the alternate forward function ... can we use
>>> "invifunc" for the inverse transition function? Or maybe the
>>> prefix should be just "i".
>
>> Hmm, I'm not a fan of any of those names. Perhaps "win" as a prefix to
>> denote a sliding window? Or just "m" for "moving aggregate".
>
> Hmm ... "moving aggregate" is actually a pretty good name for this
> whole feature -- better than "invertible aggregate" anyway. I can
> feel a global-search-and-replace coming on.
>
> So if we go with that terminology, perhaps these names for the
> new CREATE AGGREGATE parameters:
>
> initfunc applies to plain aggregation, mutually exclusive with initcond
> msfunc (or just mfunc?) forward transition for moving-agg mode
> mifunc inverse transition for moving-agg mode
> mstype state datatype for moving-agg mode
> msspace space estimate for mstype
> mfinalfunc final function for moving-agg mode
> minitfunc "firsttrans" for moving-agg mode
> minitcond mutually exclusive with minitfunc
>
Yeah, those work for me.
I think I prefer "mfunc" to "msfunc", but perhaps that's just my
natural aversion to the "ms" prefix :-)
Also, perhaps "minvfunc" rather than "mifunc" because "i" by itself
could mean "initial".
Regards,
Dean
From | Date | Subject | |
---|---|---|---|
Next Message | Sergey Muraviov | 2014-04-10 20:14:05 | Re: Problem with displaying "wide" tables in psql |
Previous Message | Alexander Korotkov | 2014-04-10 19:00:10 | Re: Partial match fix for fast scan |