On Fri, 22 Sept 2023 at 08:49, Ashutosh Bapat
<ashutosh.bapat.oss@gmail.com> wrote:
>
> Following code in ExecInterpExpr makes it clear that the
> deserialization function is be executed in per tuple memory context.
> Whereas the aggregate's context is different from this context and may
> lives longer that the context in which deserialization is expected to
> happen.
>
Right. I was about to reply, saying much the same thing, but it's
always better when you see it for yourself.
> Hence I have changed interval_avg_deserialize() in 0007 to use
> CurrentMemoryContext instead of aggcontext.
+1. And consistency with other deserialisation functions is good.
> Rest of the patches are
> same as previous set.
>
OK, I'll take a look.
Regards,
Dean