Skip site navigation (1) Skip section navigation (2)

Re: Writeable CTEs and empty relations

From: Marko Tiikkaja <marko(dot)tiikkaja(at)cs(dot)helsinki(dot)fi>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Writeable CTEs and empty relations
Date: 2010-02-10 10:05:53
Message-ID: (view raw, whole thread or download thread mbox)
Lists: pgsql-hackers
On 2010-02-10 03:20 +0200, Tom Lane wrote:
> Marko Tiikkaja <marko(dot)tiikkaja(at)cs(dot)helsinki(dot)fi> writes:
>> On 2010-02-10 02:19 +0200, Tom Lane wrote:
>>> You still haven't explained why it's a good idea to change the snapshot
>>> after the executor has started.  Right at the moment I'm prepared to
>>> reject the patch on that ground alone.
>> The patch only touches the snapshot's curcid.  That's needed to allow
>> the queries see the tuples of the DML WITHs ran before that particular
>> query.
> ... and they will also see, for example, tuples inserted by triggers
> fired by those queries.  I thought the plan for this feature was to
> store and re-read the RETURNING data, not to go back and re-read the
> underlying tables.

I originally suggested this approach about four months ago.  During this
whole time you haven't expressed any concerns about this, but suddenly
it's a reason to reject the patch?

Anyway, if we want to avoid modifying the snapshot, we can't bump the
CID between queries.  I haven't thought this through, but it seems like
this could work.  However, none of the WITH queries can see the previous
statement's tuples, which means that someone may be suprised when they
try to modify the same tuples twice just to discover that only the first
modification took place.  I don't see this as a huge problem though.

This will also solve the problem I started this thread for and makes the
patch smaller, simpler and even a bit prettier.  Unless someone sees a
problem with this approach, I'm going to make the change.

Marko Tiikkaja

In response to


pgsql-hackers by date

Next:From: Kurt HarrimanDate: 2010-02-10 10:17:32
Subject: Re: Patch: Remove gcc dependency in definition of inline functions
Previous:From: Kurt HarrimanDate: 2010-02-10 09:43:47
Subject: Re: Patch: Remove gcc dependency in definition of inline functions

Privacy Policy | About PostgreSQL
Copyright © 1996-2018 The PostgreSQL Global Development Group