Include TableFunc references when computing expression dependencies.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 13 Nov 2019 17:11:49 +0000 (12:11 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 13 Nov 2019 17:11:49 +0000 (12:11 -0500)
commit2c7b5dad6eb1602839e5961b4d4eb6494fe4a1ee
treea8b44499e64c0c680bb8162cdb0892e9c1198b5d
parent29aeda6e4e609b66b6f1c3c73b4def6fb34d90ed
Include TableFunc references when computing expression dependencies.

The TableFunc node (i.e., XMLTABLE) includes type and collation OIDs
that might not be referenced anywhere else in the expression tree,
so they need to be accounted for when extracting dependencies.

Fortunately, the practical effects of this are limited, since
(a) it's somewhat unlikely that people would be extracting
columns of non-builtin types from an XML document, and (b)
in many scenarios, the query would contain other references
to such types, or functions depending on them.  However, it's
not hard to construct examples wherein the existing code lets
one drop a type used in XMLTABLE and thereby break a view.

This is evidently an original oversight in the XMLTABLE patch,
so back-patch to v10 where that came in.

Discussion: https://postgr.es/m/18427.1573508501@sss.pgh.pa.us
src/backend/catalog/dependency.c