MIME-Version: 1.0
References: <20250718175314.4513c00a@karst>
 <CAPmGK15CQK-oYFMAyq+rR0rQapUHtvAGuGgY5ahERHzZ4tmC8g@mail.gmail.com>
 <20250729174852.14f23557@karst>
 <CAPmGK16v_We-k30qQaP+AARTr3n_dRg6yFuHP39sjV5uE_ne0Q@mail.gmail.com>
 <CAN-LCVMz58ukZ7ubGXiLuTeFE7wWmSwDw4URpF0q1ejzRvqbzg@mail.gmail.com>
 <CAN-LCVM2iOWkfFt22yVEGOrp-76YP3-BVKQg+A20TENkVh8o1w@mail.gmail.com>
 <CAN-LCVPgq0zfOU+BLrHnr2Sex_zndNjzWoAiONWD=R4ULQ2BAA@mail.gmail.com>
 <CAPmGK166P+ngd2ehady=_f-L4MePgBdBNxN5gi5_gSAfmV82QA@mail.gmail.com>
 <CAPmGK14KEFMTuQ1vYwWCo8SLks5rXv-56K-V+XMy4q8uQJvq1w@mail.gmail.com>
In-Reply-To: 
 <CAPmGK14KEFMTuQ1vYwWCo8SLks5rXv-56K-V+XMy4q8uQJvq1w@mail.gmail.com>
From: Nikita Malakhov <hukutoc@gmail.com>
Date: Sat, 6 Jun 2026 23:33:32 +0300
Message-ID: 
 <CAN-LCVPbBKPDUeoqxBwWHyaZWm5oVs4A7GD2K8daRuKAKgo4xg@mail.gmail.com>
Subject: Re: [(known) BUG] DELETE/UPDATE more than one row in partitioned
 foreign table
To: Etsuro Fujita <etsuro.fujita@gmail.com>
Cc: Jehan-Guillaume de Rorthais <jgdr@dalibo.com>,
 pgsql-hackers@lists.postgresql.org
Content-Type: multipart/alternative; boundary="0000000000009f61ee06539bb169"
Archived-At: 
 <https://www.postgresql.org/message-id/CAN-LCVPbBKPDUeoqxBwWHyaZWm5oVs4A7GD2K8daRuKAKgo4xg%40mail.gmail.com>
Precedence: bulk

--0000000000009f61ee06539bb169
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi!

Thanks for working on the subject! I'll try to take a look inside in a
couple of days.

On Fri, Jun 5, 2026 at 2:59=E2=80=AFPM Etsuro Fujita <etsuro.fujita@gmail.c=
om>
wrote:

> On Mon, Jun 1, 2026 at 7:44=E2=80=AFPM Etsuro Fujita <etsuro.fujita@gmail=
.com>
> wrote:
> > I think we should first work on a back-patchable solution.  So I'd
> > like to re-propose the patch that I proposed in this thread before to
> > disallow UPDATE/DELETE in problematic cases [1].  Attached is a new
> > version of the patch.  Changes are:
> >
> > * Renamed the new table option inherited to remotely_inherited, to
> > avoid confusion with local inheritance.
> > * Moved the logic to prevent problematic UPDATE/DELETE from a planner
> > function to an executor function, to avoid throwing an error
> > unnecessarily when there are no target rows to update/delete.
> > * Added docs to postgres-fdw.sgml.
> >
> > I'm planning to add the postgresImportForeignSchema() support in the
> > next version.
>
> I created the patch to add that support on top of the patch I sent in
> a previous email, which I'm attaching along with the base patch.  It's
> the same as before, except that I fixed a typo in docs pointed out by
> Michael-san off-list.
>
> Comments welcome!
>
> Best regards,
> Etsuro Fujita
>


--=20
Regards,
Nikita Malakhov
Postgres Professional
The Russian Postgres Company
https://postgrespro.ru/

--0000000000009f61ee06539bb169
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div dir=3D"ltr">Hi!<div><br></div><div>Thanks for working=
 on the subject! I&#39;ll try to take a look inside in a couple of days.</d=
iv></div><br><div class=3D"gmail_quote gmail_quote_container"><div dir=3D"l=
tr" class=3D"gmail_attr">On Fri, Jun 5, 2026 at 2:59=E2=80=AFPM Etsuro Fuji=
ta &lt;<a href=3D"mailto:etsuro.fujita@gmail.com">etsuro.fujita@gmail.com</=
a>&gt; wrote:<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0p=
x 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On=
 Mon, Jun 1, 2026 at 7:44=E2=80=AFPM Etsuro Fujita &lt;<a href=3D"mailto:et=
suro.fujita@gmail.com" target=3D"_blank">etsuro.fujita@gmail.com</a>&gt; wr=
ote:<br>
&gt; I think we should first work on a back-patchable solution.=C2=A0 So I&=
#39;d<br>
&gt; like to re-propose the patch that I proposed in this thread before to<=
br>
&gt; disallow UPDATE/DELETE in problematic cases [1].=C2=A0 Attached is a n=
ew<br>
&gt; version of the patch.=C2=A0 Changes are:<br>
&gt;<br>
&gt; * Renamed the new table option inherited to remotely_inherited, to<br>
&gt; avoid confusion with local inheritance.<br>
&gt; * Moved the logic to prevent problematic UPDATE/DELETE from a planner<=
br>
&gt; function to an executor function, to avoid throwing an error<br>
&gt; unnecessarily when there are no target rows to update/delete.<br>
&gt; * Added docs to postgres-fdw.sgml.<br>
&gt;<br>
&gt; I&#39;m planning to add the postgresImportForeignSchema() support in t=
he<br>
&gt; next version.<br>
<br>
I created the patch to add that support on top of the patch I sent in<br>
a previous email, which I&#39;m attaching along with the base patch.=C2=A0 =
It&#39;s<br>
the same as before, except that I fixed a typo in docs pointed out by<br>
Michael-san off-list.<br>
<br>
Comments welcome!<br>
<br>
Best regards,<br>
Etsuro Fujita<br>
</blockquote></div><div><br clear=3D"all"></div><div><br></div><span class=
=3D"gmail_signature_prefix">-- </span><br><div dir=3D"ltr" class=3D"gmail_s=
ignature"><div dir=3D"ltr"><div style=3D"color:rgb(34,34,34)">Regards,</div=
><div style=3D"color:rgb(34,34,34)">Nikita Malakhov</div><div style=3D"colo=
r:rgb(34,34,34)">Postgres Professional</div><div style=3D"color:rgb(34,34,3=
4)">The Russian Postgres Company</div><div style=3D"color:rgb(34,34,34)"><a=
 href=3D"https://postgrespro.ru/" target=3D"_blank">https://postgrespro.ru/=
</a></div></div></div></div>

--0000000000009f61ee06539bb169--