pgsql: Re-enable error for "SELECT ... OFFSET -1". - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Re-enable error for "SELECT ... OFFSET -1".
Date
Msg-id E1X9dtY-00087U-2c@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Re-enable error for "SELECT ... OFFSET -1".

The executor has thrown errors for negative OFFSET values since 8.4 (see
commit bfce56eea45b1369b7bb2150a150d1ac109f5073), but in a moment of brain
fade I taught the planner that OFFSET with a constant negative value was a
no-op (commit 1a1832eb085e5bca198735e5d0e766a3cb61b8fc).  Reinstate the
former behavior by only discarding OFFSET with a value of exactly 0.  In
passing, adjust a planner comment that referenced the ancient behavior.

Back-patch to 9.3 where the mistake was introduced.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/27048980f503da22dcd289ec8342b7021c8e73e6

Modified Files
--------------
src/backend/optimizer/plan/planner.c |    7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Re-enable error for "SELECT ... OFFSET -1".
Next
From: Noah Misch
Date:
Subject: pgsql: MSVC: Substitute $(top_builddir) in REGRESS_OPTS.