On Sun, Dec 22, 2013 at 1:01 AM, Erik Rijkers <er@xs4all.nl> wrote:
On Sat, December 21, 2013 12:52, David Rowley wrote: > On Sun, Dec 22, 2013 at 12:49 AM, Erik Rijkers <er@xs4all.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.