NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitb743a8b

committed

Fix obsolete comment in set_rel_size().

The cross-reference to set_append_rel_pathlist() was obsoleted bycommite2fa76d, which split whathad been set_rel_pathlist() and child routines into two sets offunctions. But I (tgl) evidently missed updating this comment.Back-patch to 9.2 to avoid unnecessary divergence among branches.Amit Langote

1 parentd3f5d28 commitb743a8bCopy full SHA for b743a8b

File tree

1 file changed

-1

lines changed

src/backend/optimizer/path
- allpaths.c

1 file changed

-1

lines changed

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

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -224,7 +224,7 @@ set_rel_size(PlannerInfo root, RelOptInfo rel,`
`224`	`224`	`* We proved we don't need to scan the rel via constraint exclusion,`
`225`	`225`	`* so set up a single dummy path for it. Here we only check this for`
`226`	`226`	`* regular baserels; if it's an otherrel, CE was already checked in`
`227`		`- *set_append_rel_pathlist().`
	`227`	`+ *set_append_rel_size().`
`228`	`228`	`*`
`229`	`229`	`* In this case, we go ahead and set up the relation's path right away`
`230`	`230`	`* instead of leaving it for set_rel_pathlist to do. This is because`

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!

Commitb743a8b

File tree

1 file changed

1 file changed

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

0 commit comments