NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitad480bd

committed

Ignore extra subquery outputs in set_subquery_size_estimates().

In commit0f61d4d, I added code to copy upcolumn width estimates for each column of a subquery. That code supposedthat the subquery couldn't have any output columns that didn't correspondto known columns of the current query level --- which is true when a queryis parsed from scratch, but the assumption fails when planning a view thatdepends on another view that's been redefined (adding output columns) sincethe upper view was made. This results in an assertion failure or even acrash, as per bug #8025 from lindebg. Remove the Assert and instead skipthe column if its resno is out of the expected range.

1 parent861aac5 commitad480bdCopy full SHA for ad480bd

File tree

1 file changed

-1

lines changed

src/backend/optimizer/path
- costsize.c

1 file changed

-1

lines changed

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

Lines changed: 9 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -3261,6 +3261,15 @@ set_subquery_size_estimates(PlannerInfo root, RelOptInfo rel,`
`3261`	`3261`	`if (te->resjunk)`
`3262`	`3262`	`continue;`
`3263`	`3263`
	`3264`	`+/*`
	`3265`	`+ * The subquery could be an expansion of a view that's had columns`
	`3266`	`+ * added to it since the current query was parsed, so that there are`
	`3267`	`+ * non-junk tlist columns in it that don't correspond to any column`
	`3268`	`+ * visible at our query level. Ignore such columns.`
	`3269`	`+ */`
	`3270`	`+if (te->resno<rel->min_attr\|\|te->resno>rel->max_attr)`
	`3271`	`+continue;`
	`3272`	`+`
`3264`	`3273`	`/*`
`3265`	`3274`	`* XXX This currently doesn't work for subqueries containing set`
`3266`	`3275`	`* operations, because the Vars in their tlists are bogus references`
`@@ -3282,7 +3291,6 @@ set_subquery_size_estimates(PlannerInfo root, RelOptInfo rel,`
`3282`	`3291`
`3283`	`3292`	`item_width=subrel->attr_widths[var->varattno-subrel->min_attr];`
`3284`	`3293`	`}`
`3285`		`-Assert(te->resno >=rel->min_attr&&te->resno <=rel->max_attr);`
`3286`	`3294`	`rel->attr_widths[te->resno-rel->min_attr]=item_width;`
`3287`	`3295`	`}`
`3288`	`3296`

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!

Commitad480bd

File tree

1 file changed

1 file changed

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

0 commit comments