NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit30d6a98

committed

Re-enable error for "SELECT ... OFFSET -1".

The executor has thrown errors for negative OFFSET values since 8.4 (seecommitbfce56e), but in a moment of brainfade I taught the planner that OFFSET with a constant negative value was ano-op (commit1a1832e). Reinstate theformer behavior by only discarding OFFSET with a value of exactly 0. Inpassing, adjust a planner comment that referenced the ancient behavior.Back-patch to 9.3 where the mistake was introduced.

1 parent1ea9414 commit30d6a98Copy full SHA for 30d6a98

File tree

1 file changed

-4

lines changed

src/backend/optimizer/plan
- planner.c

1 file changed

-4

lines changed

`‎src/backend/optimizer/plan/planner.c`

Lines changed: 3 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -2334,7 +2334,7 @@ preprocess_limit(PlannerInfo *root, double tuple_fraction,`
`2334`	`2334`	`{`
`2335`	`2335`	`offset_est=DatumGetInt64(((Const)est)->constvalue);`
`2336`	`2336`	`if (*offset_est<0)`
`2337`		`-offset_est=0;/less than 0 is sameas0 */`
	`2337`	`+offset_est=0;/treatasnot present */`
`2338`	`2338`	`}`
`2339`	`2339`	`}`
`2340`	`2340`	`else`
`@@ -2495,9 +2495,8 @@ limit_needed(Query *parse)`
`2495`	`2495`	`{`
`2496`	`2496`	`int64offset=DatumGetInt64(((Const*)node)->constvalue);`
`2497`	`2497`
`2498`		`-/* Executor would treat less-than-zero same as zero */`
`2499`		`-if (offset>0)`
`2500`		`-return true;/* OFFSET with a positive value */`
	`2498`	`+if (offset!=0)`
	`2499`	`+return true;/* OFFSET with a nonzero value */`
`2501`	`2500`	`}`
`2502`	`2501`	`}`
`2503`	`2502`	`else`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit30d6a98

File tree

1 file changed

1 file changed

`‎src/backend/optimizer/plan/planner.c`

0 commit comments