Dear Ajin,

> Thanks for your review Kuroda-san, I have changed the logic to not use
> already_locked and instead check if the locks are taken inside
> UpdateSubscriptionRelState itself. I've tested this and this works
> fine. If this logic is acceptable to the reviewers I can update the
> other patches also in a similar way.

Thanks for updates.
However, I found that functions like LockHeldByMe(), 
CheckRelationOidLockedByMe()
and LWLockHeldByMe() have been used only for the debug build. Functions like
ProcArraySetReplicationSlotXmin() and MarkAsPrepared() can remove the flag from
the argument but they are retained till now.
Based on that, I suggest adding new argument (or add new Ex function for bank 
branches)
and do the assertion check when the assertion is enabled in this build. Thought?

Best regards,
Hayato Kuroda
FUJITSU LIMITED

Reply via email to