Re: Assert failure in CTE inlining with view and correlated subquery
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Tomas Vondra <tomas.vondra@enterprisedb.com>
Cc: Richard Guo <guofenglinux@gmail.com>,
PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2022-04-21T21:59:33Z
Lists: pgsql-hackers
Tomas Vondra <tomas.vondra@enterprisedb.com> writes: > On 4/21/22 21:03, Tom Lane wrote: >> I think we should just drop this cross-check altogether; it is not nearly >> useful enough to justify the work that'd be involved in maintaining >> cterefcount accurately for all such transformations. All it's really >> there for is to be sure that we don't need to make a subplan for the >> inlined CTE. > WFM. I'm not particularly attached to the assert, so if you say it's not > worth it let's get rid of it. Done. regards, tom lane
Commits
-
Remove inadequate assertion check in CTE inlining.
- eade0043caaf 12.11 landed
- e6440d5007aa 13.7 landed
- da22ef388a24 14.3 landed
- 92e7a5375209 15.0 landed