pgsql: Remove gratuitous assumptions about what make_modifytable can se

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Remove gratuitous assumptions about what make_modifytable can se
Date: 2023-02-20 17:06:42
Message-ID: E1pU9cg-0027Df-9P@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Remove gratuitous assumptions about what make_modifytable can see.

For no clearly good reason, make_modifytable assumed that it
could not reach its get-the-FDW-info-the-hard-way path in MERGE.
It's currently possible to demonstrate that assertion failing,
which seems to be due to an upstream planner bug; but there's no
good reason to do it like this at all. Let's apply the principle
of separation of concerns and make the MERGE check separately,
after getting or not getting the fdwroutine pointer.

Per report from Alexander Lakhin. No test case, since I think
the potential test condition will go away soon.

Discussion: https://postgr.es/m/36bee393-b351-16ac-93b2-d46d83637e45@gmail.com

Branch
------
master

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

Modified Files
--------------
src/backend/optimizer/plan/createplan.c | 42 ++++++++++++++++-----------------
1 file changed, 20 insertions(+), 22 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Jeff Davis 2023-02-20 19:29:05 pgsql: Limit memory usage of pg_walinspect functions.
Previous Message Tom Lane 2023-02-20 15:17:41 Re: pgsql: Add assert checking to pg_leftmost_one_pos32() and friends