On Mon, 2005-02-07 at 10:41 -0500, Tom Lane wrote:
> My recollection is that I was not nearly as worried about short-term
> pallocs in the plpgsql code itself, as about leakage in various main-
> backend routines that get called incidentally during parsing.
> backend/parser/ is quite cavalier about this as a whole, because it
> expects to be called in contexts that are not long-lived.
Hmmm. What about switching the CurrentMemoryContext to the function's
long-lived context when we invoke plpgsql_yyparse(), but keeping it as
the short-lived context during the rest of the compilation process? This
unfortunately complicates the memory management model still further, but
it should significantly reduce the chance of any memory leaks.
In response to
pgsql-patches by date
|Next:||From: Tom Lane||Date: 2005-02-08 00:22:15|
|Subject: Re: WIP: pl/pgsql cleanup |
|Previous:||From: Guillaume LELARGE||Date: 2005-02-07 22:58:14|
|Subject: Updated french translations|