On Fri, Sep 11, 2026 at 10:41 AM shveta malik <shveta.malik@gmail.com> wrote:
>
> On Thu, Sep 10, 2026 at 7:40 PM Nisha Moond <nisha.moond412@gmail.com> wrote:
> >
> > On Thu, Sep 10, 2026 at 5:37 PM shveta malik <shveta.malik@gmail.com> wrote:
> > > > ~~~
> > > >
> > > > Given that we are treating a detach-pending partition as an individual
> > > > table for publication decisions, the changes in patch-003 (pg19
> > > > regression) look correct to me. The changes in relcache.c correctly
> > > > treat the detach-pending partition as an individual table.
> > > >
> > >
> > > I haven't had a chance to reveiw 003 yet as 001 itself was
> > > problematic. Will review it tomorrow.
> > >
> >
> > Here is a rebased version of 003 that applies independently on both
> > HEAD and pg19. This should make reviewing patch-003 separately easier.
> >
>
> Okay, the fix looks good. I verified that it fixes the crash on HEAD.
> But it would be good to change the fix slightly to be consistent with
> 001. See the attached changes for reference.
>
Thanks for sharing the patch. It looks good to me. I’ve updated the
attached patch accordingly.
One additional improvement:
The error message when trying to add a detach-pending partition to the
EXCEPT clause was misleading:
ERROR: cannot specify relation "public.child" in the publication EXCEPT clause
DETAIL: This operation is not supported for individual partitions.
The relation is no longer a partition in this state; it is in
detach-pending. I’ve updated the error message to make this clearer:
ERROR: cannot specify relation "public.child" in the publication EXCEPT clause
DETAIL: This operation is not supported for partitions with an
incomplete detach.
HINT: Use ALTER TABLE ... DETACH PARTITION ... FINALIZE to complete
the pending detach operation.
--
Thanks,
Nisha
