Re: Snapshot management, final - Mailing list pgsql-patches

From Simon Riggs
Subject Re: Snapshot management, final
Date
Msg-id 1208957115.4259.1312.camel@ebony.site
Whole thread Raw
In response to Re: Snapshot management, final  (Alvaro Herrera <alvherre@commandprompt.com>)
Responses Re: Snapshot management, final  (Alvaro Herrera <alvherre@commandprompt.com>)
List pgsql-patches
On Wed, 2008-04-23 at 08:21 -0400, Alvaro Herrera wrote:
> Simon Riggs wrote:
> > On Tue, 2008-04-22 at 18:13 -0400, Alvaro Herrera wrote:
> > > Simon Riggs wrote:
> > >
> > > > OK, so it can;t be copied to a longer lived memory context?
> > >
> > > If you need that ability, please explain.
> >
> > No, I wish to prevent that, not enable it.
>
> I see.  Sure, we don't have that problem.  In fact, we didn't have it
> before either, so I'm not sure I see your point :-)

You originally said "because its not needed" but didn't explain why. I
wanted to make sure there was no loophole. I'm not trying to make any
other point, just checking.

Forgive me for being dense, but what is there to stop you using a
CopySnapshot in TopMemoryContext? If you did, there would be no way to
free it, nor would we notice it had been done, AFAICS. Not anything I'm
thinking about doing, though.

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


pgsql-patches by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: Snapshot management, final
Next
From: Alvaro Herrera
Date:
Subject: Re: Snapshot management, final