NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit5e444a2

committed

Compare varnullingrels too in assign_param_for_var().

Oversight in2489d76. Preliminary analysis suggests that theproblem may be unreachable --- but if we did have instances ofthe same column with different varnullingrels, we'd surely needto treat them as different Params.Discussion:https://postgr.es/m/412552.1706203379@sss.pgh.pa.us

1 parent25cd2d6 commit5e444a2Copy full SHA for 5e444a2

File tree

1 file changed

-1

lines changed

src/backend/optimizer/util
- paramassign.c

1 file changed

-1

lines changed

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

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -90,7 +90,8 @@ assign_param_for_var(PlannerInfo root, Var var)`
`90`	`90`	`pvar->varattno==var->varattno&&`
`91`	`91`	`pvar->vartype==var->vartype&&`
`92`	`92`	`pvar->vartypmod==var->vartypmod&&`
`93`		`-pvar->varcollid==var->varcollid)`
	`93`	`+pvar->varcollid==var->varcollid&&`
	`94`	`+bms_equal(pvar->varnullingrels,var->varnullingrels))`
`94`	`95`	`returnpitem->paramId;`
`95`	`96`	`}`
`96`	`97`	`}`

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!

Commit5e444a2

File tree

1 file changed

1 file changed

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

0 commit comments