Re: Use opresulttype instead of calling SearchSysCache1() in match_orclause_to_indexcol()

Hi Tender,

> Although this would not improve performance, it can save some CPU time.
> Any thoughts?

The patch looks correct to me. The correctness can also be rechecked like this:

```
--- a/src/backend/optimizer/path/indxpath.c
+++ b/src/backend/optimizer/path/indxpath.c
@@ -3372,6 +3372,9 @@ match_orclause_to_indexcol(PlannerInfo *root,
/* Only the operator returning a boolean suit the
transformation. */
if (get_op_rettype(opno) != BOOLOID)
break;
+ else {
+ Assert(subClause->opresulttype == BOOLOID);
+ }
```

--
Best regards,
Aleksander Alekseev