NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit3f36070

committed

Fix adjust_semi_join to be more cautious about clauseless joins.

It was reporting that these were fully indexed (hence cheap), when ofcourse they're the exact opposite of that. I'm not certain if the casewould arise in practice, since a clauseless semijoin is hard to producein SQL, but if it did happen we'd make some dumb decisions.

1 parent368e44f commit3f36070Copy full SHA for 3f36070

File tree

1 file changed

+13

-5

lines changed

src/backend/optimizer/path
- costsize.c

1 file changed

+13

-5

lines changed

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

Lines changed: 13 additions & 5 deletions

Original file line number	Diff line number	Diff line change
`@@ -2810,12 +2810,20 @@ adjust_semi_join(PlannerInfo root, JoinPath path, SpecialJoinInfo *sjinfo,`
`2810`	`2810`	`*/`
`2811`	`2811`	`if (indexed_join_quals)`
`2812`	`2812`	`{`
`2813`		`-List*nrclauses;`
	`2813`	`+if (path->joinrestrictinfo!=NIL)`
	`2814`	`+{`
	`2815`	`+List*nrclauses;`
`2814`	`2816`
`2815`		`-nrclauses=select_nonredundant_join_clauses(root,`
`2816`		`-path->joinrestrictinfo,`
`2817`		`-path->innerjoinpath);`
`2818`		`-*indexed_join_quals= (nrclauses==NIL);`
	`2817`	`+nrclauses=select_nonredundant_join_clauses(root,`
	`2818`	`+path->joinrestrictinfo,`
	`2819`	`+path->innerjoinpath);`
	`2820`	`+*indexed_join_quals= (nrclauses==NIL);`
	`2821`	`+}`
	`2822`	`+else`
	`2823`	`+{`
	`2824`	`+/* a clauseless join does NOT qualify */`
	`2825`	`+*indexed_join_quals= false;`
	`2826`	`+}`
`2819`	`2827`	`}`
`2820`	`2828`
`2821`	`2829`	`return true;`

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!

Commit3f36070

File tree

1 file changed

1 file changed

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

0 commit comments