Re: [PATCH] Negative Transition Aggregate Functions (WIP)

On Sun, Dec 22, 2013 at 2:42 AM, David Rowley <dgrowleyml(at)gmail(dot)com> wrote:

> On Sun, Dec 22, 2013 at 1:01 AM, Erik Rijkers <er(at)xs4all(dot)nl> wrote:
>
>> On Sat, December 21, 2013 12:52, David Rowley wrote:
>> > On Sun, Dec 22, 2013 at 12:49 AM, Erik Rijkers <er(at)xs4all(dot)nl> wrote:
>> >
>> >> On Sat, December 21, 2013 12:38, David Rowley wrote:
>> >> > [ inverse_transition_functions_v1.2.patch.gz ]
>> >> > Please find attached an updated patch which should remove the
>> duplicate
>> >> OID
>> >> > problem you saw.
>> >>
>> >> That fixes it, thanks
>> >>
>> >> There is 1 of 141 failed tests:
>> >>
>> >> window ... FAILED
>> >>
>> >>
>> > That's strange, it passes here.
>> >
>> > Would you be able to send me the regression diff file?
>> >
>>
>> attached...
>>
>
> Thanks
>
> This was just down to some missing trailing white space on the expected
> results. I've fixed these up and attached another patch.
>
>
I've attached an updated patch which is re-merged to current head.

I've also done a bit more work to the patch:
Added inverse transition functions for all the stddev() type aggregates for
all int types.
Added inverse transition function for SUM with bigint
Added pg_dump support.

I've yet to do SUM or AVG for numeric types due to some concern with having
more digits after the decimal point after inverse transitions take place.

Regards

David Rowley

Regards
>
> David Rowley
>
>