]> granicus.if.org Git - postgresql/commit
Repair memory leakage while ANALYZE-ing complex index expressions.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 9 Nov 2010 16:28:18 +0000 (11:28 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 9 Nov 2010 16:51:19 +0000 (11:51 -0500)
commit1d5edff1b0a67fe75946ba8ff58d93b13de77e54
tree7b9b0e6630b495f01b017b5fba1e6179745a845b
parent17b38fae86f07355efa5d7dcaef345c60c88f69a
Repair memory leakage while ANALYZE-ing complex index expressions.

The general design of memory management in Postgres is that intermediate
results computed by an expression are not freed until the end of the tuple
cycle.  For expression indexes, ANALYZE has to re-evaluate each expression
for each of its sample rows, and it wasn't bothering to free intermediate
results until the end of processing of that index.  This could lead to very
substantial leakage if the intermediate results were large, as in a recent
example from Jakub Ouhrabka.  Fix by doing ResetExprContext for each sample
row.  This necessitates adding a datumCopy step to ensure that the final
expression value isn't recycled too.  Some quick testing suggests that this
change adds at worst about 10% to the time needed to analyze a table with
an expression index; which is annoying, but seems a tolerable price to pay
to avoid unexpected out-of-memory problems.

Back-patch to all supported branches.
src/backend/commands/analyze.c