On Tue, 2008-04-22 at 15:49 -0400, Alvaro Herrera wrote:

> - Three CopySnapshot call sites remain outside snapmgr.c: DoCopy() on
> copy.c, ExplainOnePlan() on explain.c and _SPI_execute_plan() on spi.c.
> They are there because they grab the current ActiveSnapshot, modify it,
> and then use the resulting snapshot.  There is no corresponding
> FreeSnapshot, because it's not needed.

Not needed? How can we be certain that the modified snapshot does not
outlive its original source?

-- 
  Simon Riggs
  2ndQuadrant  http://www.2ndQuadrant.com


-- 
Sent via pgsql-patches mailing list (pgsql-patches@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-patches

Reply via email to