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(-)
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 |