Re: Combine function returning NULL unhandled?

2017-11-30 Thread David Rowley
On 24 November 2017 at 14:20, Andres Freund wrote: > Pushed a fix to the relevant branches, including tests of the > trans/combine functions returns NULL cases. Apologies for my silence here. I've been on leave and out of internet range for two weeks. Thank you for making

Re: Combine function returning NULL unhandled?

2017-11-23 Thread Andres Freund
On 2017-11-21 15:51:59 -0500, Robert Haas wrote: > On Mon, Nov 20, 2017 at 10:36 PM, Andres Freund wrote: > > The plain transition case contains: > > if (pergroupstate->transValueIsNull) > > { > > /* > >

Re: Combine function returning NULL unhandled?

2017-11-21 Thread Robert Haas
On Mon, Nov 20, 2017 at 10:36 PM, Andres Freund wrote: > The plain transition case contains: > if (pergroupstate->transValueIsNull) > { > /* > * Don't call a strict function with NULL inputs. >