NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit834d97c

committed

Fix broken MemoizePath support in reparameterize_path().

It neglected to recurse to the subpath, meaning you'd get backa path identical to the input. This could produce wrong queryresults if the omission meant that the subpath fails to enforcesome join clause it should be enforcing. We don't have a testcase for this at the moment, but the code is obviously brokenand the fix is equally obvious. Back-patch to v14 whereMemoize was introduced.Richard GuoDiscussion:https://postgr.es/m/CAMbWs4_R=ORpz=Lkn2q3ebPC5EuWyfZF+tmfCPVLBVK5W39mHA@mail.gmail.com

1 parentbf8fd64 commit834d97cCopy full SHA for 834d97c

File tree

1 file changed

-1

lines changed

src/backend/optimizer/util
- pathnode.c

1 file changed

-1

lines changed

`‎src/backend/optimizer/util/pathnode.c`

Lines changed: 7 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -3952,9 +3952,15 @@ reparameterize_path(PlannerInfo root, Path path,`
`3952`	`3952`	`caseT_Memoize:`
`3953`	`3953`	`{`
`3954`	`3954`	`MemoizePathmpath= (MemoizePath)path;`
	`3955`	`+Path*spath=mpath->subpath;`
`3955`	`3956`
	`3957`	`+spath=reparameterize_path(root,spath,`
	`3958`	`+required_outer,`
	`3959`	`+loop_count);`
	`3960`	`+if (spath==NULL)`
	`3961`	`+returnNULL;`
`3956`	`3962`	`return (Path*)create_memoize_path(root,rel,`
`3957`		`-mpath->subpath,`
	`3963`	`+spath,`
`3958`	`3964`	`mpath->param_exprs,`
`3959`	`3965`	`mpath->hash_operators,`
`3960`	`3966`	`mpath->singlerow,`

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!

Commit834d97c

File tree

1 file changed

1 file changed

`‎src/backend/optimizer/util/pathnode.c`

0 commit comments