Commit86e4ebb

committed

Allow rel_is_distinct_for() to look through RelabelType below OpExpr.

This lets it do the right thing for, eg, varchar columns.Back-patch to 9.5 where this logic appeared.David Rowley, per report from Kim Rose CarlsenDiscussion:https://postgr.es/m/VI1PR05MB17091F9A9876528055D6A827C76D0@VI1PR05MB1709.eurprd05.prod.outlook.com

1 parentc0d21bd commit86e4ebbCopy full SHA for 86e4ebb

File tree

-0

lines changed

-0

lines changed

Lines changed: 8 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -617,6 +617,14 @@ rel_is_distinct_for(PlannerInfo root, RelOptInfo rel, List *clause_list)`
`617`	`617`	`else`
`618`	`618`	`var= (Var*)get_leftop(rinfo->clause);`
`619`	`619`
	`620`	`+/*`
	`621`	`+ * We may ignore any RelabelType node above the operand. (There`
	`622`	`+ * won't be more than one, since eval_const_expressions() has been`
	`623`	`+ * applied already.)`
	`624`	`+ */`
	`625`	`+if (var&&IsA(var,RelabelType))`
	`626`	`+var= (Var) ((RelabelType)var)->arg;`
	`627`	`+`
`620`	`628`	`/*`
`621`	`629`	`* If inner side isn't a Var referencing a subquery output column,`
`622`	`630`	`* this clause doesn't help us.`

Comments

(0)