Re: FIX : teach expression walker about RestrictInfo

Tomas Vondra <tomas.vondra@2ndquadrant.com> writes:
> the attached trivial patch adds handling of RestrictInfo nodes into 
> expression_tree_walker().

RestrictInfo is not a general expression node and support for it has
been deliberately omitted from expression_tree_walker().  So I think
what you are proposing is a bad idea and probably a band-aid for some
other bad idea.

> This is needed for example when calling 
> pull_varnos or (or other functions using the expression walker) in 
> clausesel.c, for example. An example of a query causing errors with 
> pull_varnos is

> select * from t where (a >= 10 and a <= 20) or (b >= 15 and b <= 20);

Really?

regression=# create table t (a int, b int);
CREATE TABLE
regression=# select * from t where (a >= 10 and a <= 20) or (b >= 15 and b <= 20);a | b 
---+---
(0 rows)
        regards, tom lane