Re: Assertion failure in GetSubscriptionRelations() with concurrent DROP TABLE

[vignesh C]

On Tue, 25 Aug 2026 at 19:11, Amit Kapila <amit.kapil...@gmail.com> wrote:
>
> On Tue, Aug 25, 2026 at 5:39 PM Hayato Kuroda (Fujitsu)
> <kuroda.hay...@fujitsu.com> wrote:
> >
> > Thanks for updating the patch. I confirmed it could fix the issue.
> >
> >
> > ```
> > +               else
> > +                       elog(ERROR, "unexpected relkind: %d", (int) 
> > relkind);
> > ```
> >
> >
> > I feel \"%c\" might be better than %d, like what ProcessUtilitySlow() does,
> > because the relkind is defined as char.
> >
>
> Agreed, though %d is used for the same purpose at one place in code
> but other uses %c for relkind. How about a slightly more informative
> message like: elog(ERROR, "unexpected relkind \"%c\" for relation %u
> in subscription %u", relkind, subrel->srrelid, subid)?

Here is an updated version with the message updated accordingly.

Regards,
Vignesh

v3-0001-Skip-relations-dropped-concurrently-in-GetSubscri.patch
Description: Binary data