pgsql: Disallow SELECT FOR UPDATE/SHARE on sequences. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Disallow SELECT FOR UPDATE/SHARE on sequences.
Date
Msg-id E1QSCvE-0005Xb-Iq@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Disallow SELECT FOR UPDATE/SHARE on sequences.

We can't allow this because such an operation stores its transaction XID
into the sequence tuple's xmax.  Because VACUUM doesn't process sequences
(and we don't want it to start doing so), such an xmax value won't get
frozen, meaning it will eventually refer to nonexistent pg_clog storage,
and even wrap around completely.  Since the row lock is ignored by nextval
and setval, the usefulness of the operation is highly debatable anyway.
Per reports of trouble with pgpool 3.0, which had ill-advisedly started
using such commands as a form of locking.

In HEAD, also disallow SELECT FOR UPDATE/SHARE on toast tables.  Although
this does work safely given the current implementation, there seems no
good reason to allow it.  I refrained from changing that behavior in
back branches, however.

Branch
------
REL8_4_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/23138a5203fe14d6adf5019482f341ebfe2136ad

Modified Files
--------------
src/backend/executor/execMain.c |   37 +++++++++++++++++++++++++++++++++++++
1 files changed, 37 insertions(+), 0 deletions(-)


pgsql-committers by date:

Previous
From: Robert Haas
Date:
Subject: pgsql: Avoid creating init fork for unlogged indexes when it already ex
Next
From: Tom Lane
Date:
Subject: pgsql: Disallow SELECT FOR UPDATE/SHARE on sequences.