Commitbe98a55

committed

Update comment in set_dummy_rel_pathlist().

This comment claimed that set_dummy_rel_pathlist() has callersother than (possibly indirectly) set_rel_size(). It doesn't,so revise the argument to not rely on that.Noted by Richard Guo.Discussion:https://postgr.es/m/CAMbWs4-KFEU_fDuJPNCOkUu3rwvZvKBEytkd9VrM4kH4-2h1CQ@mail.gmail.com

1 parent213c959 commitbe98a55Copy full SHA for be98a55

File tree

-3

lines changed

-3

lines changed

Lines changed: 3 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -2180,9 +2180,9 @@ set_dummy_rel_pathlist(RelOptInfo *rel)`
`2180`	`2180`
`2181`	`2181`	`/*`
`2182`	`2182`	`* We set the cheapest-path fields immediately, just in case they were`
`2183`		`- * pointing at some discarded path. This is redundantwhen we're called`
`2184`		`- *from set_rel_size(), but not when called from elsewhere, and doing it`
`2185`		`- *twice is harmless anyway.`
	`2183`	`+ * pointing at some discarded path. This is redundantin current usage`
	`2184`	`+ *because set_rel_pathlist will do it later, but it's cheap so we keep it`
	`2185`	`+ *for safety and consistency with mark_dummy_rel.`
`2186`	`2186`	`*/`
`2187`	`2187`	`set_cheapest(rel);`
`2188`	`2188`	`}`

Comments

(0)