Alvaro Herrera <alvhe...@alvh.no-ip.org> writes: > On 2023-Apr-24, Tom Lane wrote: >> I guess we could have the back branches continue to create a >> shared_cast_context and just not use it in core. Seems rather >> expensive for a very hypothetical compatibility measure, though.
> I think a session-long memory leak is not so bad, compared to a possible > crash. However, after looking at the code again, as well as pldebugger > and plpgsql_check, I agree that there's no point in doing anything other > than keeping the field there. Yeah, I can't see any plausible reason for outside code to be using that field (and I don't see any evidence in Debian Code Search that anyone is). I'll push it like this then. Thanks for looking! regards, tom lane