Commit3edec38

committed

Fix Assert that's no longer correct now that RowCompareExpr is indexable.

1 parenta1b7e70 commit3edec38Copy full SHA for 3edec38

File tree

-8

lines changed

-8

lines changed

Lines changed: 4 additions & 8 deletions

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`*`
`10`	`10`	`*`
`11`	`11`	`* IDENTIFICATION`
`12`		`- * $PostgreSQL: pgsql/src/backend/optimizer/path/indxpath.c,v 1.199 2006/01/29 17:27:42 tgl Exp $`
	`12`	`+ * $PostgreSQL: pgsql/src/backend/optimizer/path/indxpath.c,v 1.200 2006/01/29 17:40:00 tgl Exp $`
`13`	`13`	`*`
`14`	`14`	`*-------------------------------------------------------------------------`
`15`	`15`	`*/`
`@@ -1611,11 +1611,6 @@ identify_ignorable_ordering_cols(PlannerInfo *root,`
`1611`	`1611`	`boolvaronleft;`
`1612`	`1612`	`boolispc;`
`1613`	`1613`
`1614`		`-/*`
`1615`		`- * We know this clause passed match_clause_to_indexcol as a`
`1616`		`- * toplevel clause; so it's not a ScalarArrayOp.`
`1617`		`- */`
`1618`		`-`
`1619`	`1614`	`/* First check for boolean-index cases. */`
`1620`	`1615`	`if (IsBooleanOpclass(opclass))`
`1621`	`1616`	`{`
`@@ -1632,8 +1627,9 @@ identify_ignorable_ordering_cols(PlannerInfo *root,`
`1632`	`1627`	`}`
`1633`	`1628`	`}`
`1634`	`1629`
`1635`		`-/* Else clause must be a binary opclause. */`
`1636`		`-Assert(IsA(clause,OpExpr));`
	`1630`	`+/* Otherwise, ignore if not a binary opclause */`
	`1631`	`+if (!is_opclause(clause)\|\|list_length(clause->args)!=2)`
	`1632`	`+continue;`
`1637`	`1633`
`1638`	`1634`	`/* Determine left/right sides and check the operator */`
`1639`	`1635`	`clause_op=clause->opno;`

Comments

(0)