Postgres Pro
Downloads
Home   >   mailing lists

Re: [PATCH] Negative Transition Aggregate Functions (WIP) - Mailing list pgsql-hackers

From David Rowley
Subject Re: [PATCH] Negative Transition Aggregate Functions (WIP)
Date
Msg-id CAApHDvo6x_qHQcOF4DMm-3A1gb8Rx-=fZRyMPcg3U02=UcRGiA@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] Negative Transition Aggregate Functions (WIP)  (David Rowley <dgrowleyml@gmail.com>)
Responses Re: [PATCH] Negative Transition Aggregate Functions (WIP)
List pgsql-hackers
Tree view
On Tue, Dec 17, 2013 at 10:51 PM, David Rowley <dgrowleyml@gmail.com> wrote:
On Mon, Dec 16, 2013 at 9:36 PM, Hannu Krosing <hannu@2ndquadrant.com> wrote:
On 12/16/2013 08:39 AM, David Rowley wrote:
 
Any other ideas or +1's for any of the existing ones?
+1, inverse good :)


In the attached patch I've renamed negative to inverse. I've also disabled the inverse functions when an expression in an aggregate contains a volatile function.


I've attached an updated patch in which I've done some more work to the documents and also added some more sanity checking around CREATE AGGREGATE. The previous patch allowed CREATE AGGREGATE to accept a transition function which was strict and an inverse transition function which was not, or vise versa.

I've also added a bunch of regression tests around create aggregate which were missing in the previous patch.

Regards

David Rowley 

Attachment

pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: shared memory message queues
Next
From: "Erik Rijkers"
Date:
Subject: Re: [PATCH] Negative Transition Aggregate Functions (WIP)