]> granicus.if.org Git - postgresql/commit
Re-enable error for "SELECT ... OFFSET -1".
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 22 Jul 2014 17:30:01 +0000 (13:30 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 22 Jul 2014 17:30:14 +0000 (13:30 -0400)
commit6306d07122d8b6678f47c273165540de02a0d242
tree1598a69d5b07d7faa498b995752569dffb9e3c68
parentf59c8eff7dd3dea4116a34e1890ee219f6d7f8d5
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.
src/backend/optimizer/plan/planner.c