NotificationsYou must be signed in to change notification settings
Fork5
Star26

Commitc59d8dd

committed

Improve pull_up_subqueries logic so that it doesn't insert unnecessary

PlaceHolderVar nodes in join quals appearing in or below the lowestouter join that could null the subquery being pulled up. This improvesthe planner's ability to recognize constant join quals, and probablyhelps with detection of common sort keys (equivalence classes) as well.

1 parent636edd5 commitc59d8ddCopy full SHA for c59d8dd

File tree

3 files changed

+148

-71

lines changed

src
- backend/optimizer
  - plan
    - planner.c
  - prep
    - prepjointree.c
- include/optimizer
  - prep.h

3 files changed

+148

-71

lines changed

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

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`*`
`9`	`9`	`*`
`10`	`10`	`* IDENTIFICATION`
`11`		`- * $PostgreSQL: pgsql/src/backend/optimizer/plan/planner.c,v 1.254 2009/04/19 19:46:33 tgl Exp $`
	`11`	`+ * $PostgreSQL: pgsql/src/backend/optimizer/plan/planner.c,v 1.255 2009/04/28 21:31:16 tgl Exp $`
`12`	`12`	`*`
`13`	`13`	`*-------------------------------------------------------------------------`
`14`	`14`	`*/`
`@@ -324,7 +324,7 @@ subquery_planner(PlannerGlobal glob, Query parse,`
`324`	`324`	`* this query.`
`325`	`325`	`*/`
`326`	`326`	`parse->jointree= (FromExpr*)`
`327`		`-pull_up_subqueries(root, (Node*)parse->jointree,false, false);`
	`327`	`+pull_up_subqueries(root, (Node*)parse->jointree,NULL,NULL);`
`328`	`328`
`329`	`329`	`/*`
`330`	`330`	`* Detect whether any rangetable entries are RTE_JOIN kind; if not, we can`

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!

Commitc59d8dd

File tree

3 files changed

3 files changed

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

0 commit comments