Re: Review: Fix snapshot taking inconsistencies - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Review: Fix snapshot taking inconsistencies
Date
Msg-id 27524.1298919675@sss.pgh.pa.us
Whole thread Raw
In response to Re: Review: Fix snapshot taking inconsistencies  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: Review: Fix snapshot taking inconsistencies  (Robert Haas <robertmhaas@gmail.com>)
List pgsql-hackers
Robert Haas <robertmhaas@gmail.com> writes:
> On Mon, Feb 28, 2011 at 1:45 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> I'm afraid that the goals of this patch might be similarly obsolete.

> No, I don't think so.  IIUC, the problem is that EXPLAIN ANALYZE runs
> the rewrite products with different snapshot handling than the regular
> execution path.

Possibly, but it's not clear to me that this patch fixes that.
As I said, it's no longer obvious what the patch means to do, and I'd
like a clear statement of that.

> So in theory you could turn on auto_explain and have
> the semantics of your queries change.  That would be Bad.

That's just FUD.  auto_explain doesn't run EXPLAIN ANALYZE.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Marko Tiikkaja
Date:
Subject: Re: Review: Fix snapshot taking inconsistencies
Next
From: Tom Lane
Date:
Subject: Re: Review: Fix snapshot taking inconsistencies