On Tue, Mar 22, 2022 at 10:28 AM Dilip Kumar <dilipbalaut@gmail.com> wrote:
>
>
> I think this make sense. I haven't changed the original patch as you
> told you were improving on some comments, so in order to avoid
> conflict I have created this add on patch.
>
In my previous patch mistakenly I used src_dboid instead of
dest_dboid. Fixed in this version. For destination db I have used
lock mode as AccessSharedLock. Logically if we see access wise we
don't want anyone else to be accessing that db but that is anyway
protected because it is not visible to anyone else. So I think
AccessSharedLock should be correct here because we are just taking
this lock because we are accessing pages in shared buffers from this
database's relations.
--
Regards,
Dilip Kumar
EnterpriseDB: http://www.enterprisedb.com