Dear Ajin,

> I have added "Ex functions" for back branches (PG_17 and earlier) ,
> which also have Asserts for making sure locks are taken. For PG_18 and
> HEAD, I've used the extra parameter already_locked.
> PG_14_15-v6-0001-Fix-a-deadlock-during-ALTER-SUBSCRIPTION.-DROP-PU.p
> atch
> is for both PG_14 and PG_15 and
> PG_18_HEAD-v6-0001-Fix-a-possible-deadlock-during-ALTER-SUBSCRIPTION
> is for both PG_18 and HEAD.
>

Thanks for creating the patch!
I feel the existing function here should be the wrapper of Ex function, so 
attached
patch did not match the expectation. table_open() and table_close() are called
both in UpdateSubscriptionRelState() and UpdateSubscriptionRelStateEx().
Another issue is that the variable "tag" is used by SET_LOCKTAG_OBJECT() even
without the debug build.

How do you feel the .diff file can be applied atop PG17 patch? It is mainly
same as v4 patch but has some assertion.

Best regards,
Hayato Kuroda
FUJITSU LIMITED

Attachment: kuroda.diffs
Description: kuroda.diffs

Reply via email to