Commit6f44fe7

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 parent27c6619 commit6f44fe7Copy full SHA for 6f44fe7

File tree

-0

lines changed

-0

lines changed

Lines changed: 8 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -703,6 +703,14 @@ rel_is_distinct_for(PlannerInfo root, RelOptInfo rel, List *clause_list)`
`703`	`703`	`else`
`704`	`704`	`var= (Var*)get_leftop(rinfo->clause);`
`705`	`705`
	`706`	`+/*`
	`707`	`+ * We may ignore any RelabelType node above the operand. (There`
	`708`	`+ * won't be more than one, since eval_const_expressions() has been`
	`709`	`+ * applied already.)`
	`710`	`+ */`
	`711`	`+if (var&&IsA(var,RelabelType))`
	`712`	`+var= (Var) ((RelabelType)var)->arg;`
	`713`	`+`
`706`	`714`	`/*`
`707`	`715`	`* If inner side isn't a Var referencing a subquery output column,`
`708`	`716`	`* this clause doesn't help us.`

Comments

(0)