On Wed, May 02, 2018 at 05:20:37PM -0400, Tom Lane wrote: > Peter Eisentraut <peter.eisentr...@2ndquadrant.com> writes: > > Yes, that was the idea. Here is an adjusted patch. > > Looks OK to me as far as the leak issue goes. I have no opinion on > whether this is adequate in respect to cleanup-after-error problems.
With connection poolers letting the connections to the server be around for a long time, wouldn't it be an issue to let this much memory live longer than the transaction context? The deeper the stack, the more memory consumed, hence the more OS cache that PostgreSQL cannot use. So this could impact performance for some loads. I would vote for cleaning up this memory instead of letting it live unused in TopMemoryContext. -- Michael
signature.asc
Description: PGP signature
