pgsql: Re-allow INDEX_VAR as rt_index in ChangeVarNodes(). - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Re-allow INDEX_VAR as rt_index in ChangeVarNodes().
Date
Msg-id E1q7JB1-000mtz-8U@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Re-allow INDEX_VAR as rt_index in ChangeVarNodes().

Apparently some extensions are in the habit of calling
ChangeVarNodes() with INDEX_VAR as the rt_index to replace.
That worked before 2489d76c4, at least as long as there were
not PlaceHolderVars in the expression; but now it fails
because bms_is_member spits up.  Add a test to avoid that.

Per report from Anton Melnikov, though this is not his
proposed patch.

Discussion: https://postgr.es/m/5b370a46-f6d2-373d-9dbc-0d55250e82c1@inbox.ru

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/fbf80421ead55deaafbefa808960a2695de492c9

Modified Files
--------------
src/backend/rewrite/rewriteManip.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Fix small overestimation of base64 encoding output length.
Next
From: Tom Lane
Date:
Subject: pgsql: Fix oversight in outer join removal.