database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-05-18 17:41:14 +03:00
Code

Fix oversights in processing of LIMIT expressions during planning.

Browse Source
This commit is contained in:
Tom Lane 2004-05-11 13:15:23 +00:00
parent 40ac7eddd9
commit b8de9b216f
2 changed files with 18 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/backend/optimizer/plan/setrefs.c
View File

@ -9,7 +9,7 @@
* *
* *
* IDENTIFICATION * IDENTIFICATION
* $Header: /cvsroot/pgsql/src/backend/optimizer/plan/setrefs.c,v 1.97 2003/08/08 21:41:50 momjian Exp $ * $Header: /cvsroot/pgsql/src/backend/optimizer/plan/setrefs.c,v 1.97.4.1 2004/05/11 13:15:23 tgl Exp $
* *
*------------------------------------------------------------------------- *-------------------------------------------------------------------------
*/ */
@ -189,7 +189,6 @@ set_plan_references(Plan *plan, List *rtable)
case T_Sort: case T_Sort:
case T_Unique: case T_Unique:
case T_SetOp: case T_SetOp:
case T_Limit:
/* /*
* These plan types don't actually bother to evaluate their * These plan types don't actually bother to evaluate their
@ -202,6 +201,15 @@ set_plan_references(Plan *plan, List *rtable)
* the plan tree! * the plan tree!
*/ */
break; break;
case T_Limit:
/*
* Like the plan types above, Limit doesn't evaluate its
* tlist or quals. It does have live expressions for
* limit/offset, however.
*/
fix_expr_references(plan, ((Limit *) plan)->limitOffset);
fix_expr_references(plan, ((Limit *) plan)->limitCount);
break;
case T_Agg: case T_Agg:
case T_Group: case T_Group:
set_uppernode_references(plan, (Index) 0); set_uppernode_references(plan, (Index) 0);

10
src/backend/optimizer/plan/subselect.c
View File

@ -7,7 +7,7 @@
* Portions Copyright (c) 1994, Regents of the University of California * Portions Copyright (c) 1994, Regents of the University of California
* *
* IDENTIFICATION * IDENTIFICATION
* $Header: /cvsroot/pgsql/src/backend/optimizer/plan/subselect.c,v 1.83.2.1 2003/11/25 23:59:32 tgl Exp $ * $Header: /cvsroot/pgsql/src/backend/optimizer/plan/subselect.c,v 1.83.2.2 2004/05/11 13:15:23 tgl Exp $
* *
*------------------------------------------------------------------------- *-------------------------------------------------------------------------
*/ */
@ -1018,6 +1018,13 @@ finalize_plan(Plan *plan, List *rtable,
&context); &context);
break; break;
case T_Limit:
finalize_primnode(((Limit *) plan)->limitOffset,
&context);
finalize_primnode(((Limit *) plan)->limitCount,
&context);
break;
case T_Hash: case T_Hash:
finalize_primnode((Node *) ((Hash *) plan)->hashkeys, finalize_primnode((Node *) ((Hash *) plan)->hashkeys,
&context); &context);
@ -1029,7 +1036,6 @@ finalize_plan(Plan *plan, List *rtable,
case T_Sort: case T_Sort:
case T_Unique: case T_Unique:
case T_SetOp: case T_SetOp:
case T_Limit:
case T_Group: case T_Group:
break; break;