NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit926e8a0

committed

Add a back-link from IndexOptInfo structs to their parent RelOptInfo

structs. There are many places in the planner where we were passingboth a rel and an index to subroutines, and now need only pass theindex struct. Notationally simpler, and perhaps a tad faster.

1 parentfebc9a6 commit926e8a0Copy full SHA for 926e8a0

File tree

11 files changed

+95

-141

lines changed

src
- backend/optimizer
  - path
  - plan
    - createplan.c
  - util
    - pathnode.c
    - plancat.c
- include
  - nodes
    - relation.h
  - optimizer

11 files changed

+95

-141

lines changed

`‎src/backend/optimizer/path/costsize.c`

Lines changed: 2 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@`
`49`	`49`	`* Portions Copyright (c) 1994, Regents of the University of California`
`50`	`50`	`*`
`51`	`51`	`* IDENTIFICATION`
`52`		`- * $PostgreSQL: pgsql/src/backend/optimizer/path/costsize.c,v 1.138 2005/03/06 22:15:04 tgl Exp $`
	`52`	`+ * $PostgreSQL: pgsql/src/backend/optimizer/path/costsize.c,v 1.139 2005/03/27 06:29:35 tgl Exp $`
`53`	`53`	`*`
`54`	`54`	`*-------------------------------------------------------------------------`
`55`	`55`	`*/`
`@@ -227,7 +227,6 @@ cost_nonsequential_access(double relpages)`
`227`	`227`	`* but here we consider the cost of just one pass.`
`228`	`228`	`*`
`229`	`229`	`* 'root' is the query root`
`230`		`- * 'baserel' is the base relation the index is for`
`231`	`230`	`* 'index' is the index to be used`
`232`	`231`	`* 'indexQuals' is the list of applicable qual clauses (implicit AND semantics)`
`233`	`232`	`* 'is_injoin' is T if we are considering using the index scan as the inside`
`@@ -243,11 +242,11 @@ cost_nonsequential_access(double relpages)`
`243`	`242`	`*/`
`244`	`243`	`void`
`245`	`244`	`cost_index(Pathpath,Queryroot,`
`246`		`-RelOptInfo*baserel,`
`247`	`245`	`IndexOptInfo*index,`
`248`	`246`	`List*indexQuals,`
`249`	`247`	`boolis_injoin)`
`250`	`248`	`{`
	`249`	`+RelOptInfo*baserel=index->rel;`
`251`	`250`	`Coststartup_cost=0;`
`252`	`251`	`Costrun_cost=0;`
`253`	`252`	`CostindexStartupCost;`

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!

Commit926e8a0

File tree

11 files changed

11 files changed

`‎src/backend/optimizer/path/costsize.c`

0 commit comments