On Thu, 31 Jul 2025 at 08:23, Ajin Cherian <itsa...@gmail.com> wrote:
>
> On Wed, Jul 30, 2025 at 10:33 PM Hayato Kuroda (Fujitsu)
> <kuroda.hay...@fujitsu.com> wrote:
> >
> > Dear Ajin,
> >
> > > Attaching the updated patches with the changes you requested. I've
> > > also added the unchanged patches for PG_18 and HEAD (PG_18_HEAD-v6*),
> > > so that everything is together in one mail.
> >
> > Thanks for update, but the patch for PG18/HEAD seemed not to have Assert().
> > Can you modify like others do?
> >
>
> Updated patch for PG18/HEAD.
How about we change the below:
+#ifdef USE_ASSERT_CHECKING
+ LOCKTAG tag;
+#endif
+
+ Assert(CheckRelationOidLockedByMe(SubscriptionRelRelationId,
+
RowExclusiveLock, true));
+
+ rel = table_open(SubscriptionRelRelationId, NoLock);
+#ifdef USE_ASSERT_CHECKING
+ SET_LOCKTAG_OBJECT(tag, InvalidOid,
SubscriptionRelationId, subid, 0);
+ Assert(LockHeldByMe(&tag, AccessShareLock, true));
+#endif
to:
#ifdef USE_ASSERT_CHECKING
LOCKTAG tag;
Assert(CheckRelationOidLockedByMe(SubscriptionRelRelationId,
RowExclusiveLock, true));
SET_LOCKTAG_OBJECT(tag, InvalidOid, SubscriptionRelationId, subid, 0);
Assert(LockHeldByMe(&tag, AccessShareLock, true));
#endif
rel = table_open(SubscriptionRelRelationId, NoLock);
}
The rest looks good to me.
Regards,
Vignesh
