Why does exprCollation reject List node? - Mailing list pgsql-hackers

From Shigeru HANADA
Subject Why does exprCollation reject List node?
Date
Msg-id 4F631A9A.2020209@gmail.com
Whole thread Raw
Responses Re: Why does exprCollation reject List node?
List pgsql-hackers
Hi all,

During writing pgsql_fdw codes, I noticed that exprCollation rejects
non-Expr nodes with error "unrecognized node type: %d".  Is this
intentional behavior, or can it return InvalidOid for unrecognized nodes
like exprInputCollation?

Background information: I use exprCollation with expression_walker in
pgsql_fdw to know whether an expression in baserestrictinfo->clause list
uses any collation, to determine the clause can be pushed down safely.
I want to allow pushing ScalarArrayOpExpr down, but its argument is
represented as List, so a query contains ScalarArrayOpExpr  ends with
error when the traversing expression tree reaches arguments of
ScalarArrayOpExpr.

I've spent only few hours for research though, but the interface of
exprCollation seems little odd.  It accepts Node*, but its switch
statement assumes that given object should be something derived from
Expr, and it rejects other objects with elog(ERROR).

Anyone know the reason?

Regards,
-- 
Shigeru Hanada


pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: initdb and fsync
Next
From: Dimitri Fontaine
Date:
Subject: Re: Command Triggers, v16