Andres Freund <and...@anarazel.de> writes:
> On 2021-03-16 20:50:17 -0700, Andres Freund wrote:
>> What I meant was that I didn't understand how there's not a leak
>> danger when compilation fails halfway through, given that the context
>> in question is below TopMemoryContext and that I didn't see a relevant
>> TRY block. But that probably there is something cleaning it up that I
>> didn't see.
> Looks like it's an actual leak:
Yeah, I believe that. On the other hand, I'm not sure that such cases
represent any real problem for production usage. I'm inclined to focus
on non-error scenarios first.
(Having said that, we probably have the ability to fix such things
relatively painlessly now, by reparenting an initially-temporary
context once we're done parsing.)
Meanwhile, I'm still trying to understand why valgrind is whining
about the rd_indexcxt identifier strings. AFAICS it shouldn't.
regards, tom lane
