Re: locking [user] catalog tables vs 2pc vs logical rep

From: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
To: "osumi(dot)takamichi(at)fujitsu(dot)com" <osumi(dot)takamichi(at)fujitsu(dot)com>
Cc: Simon Riggs <simon(dot)riggs(at)enterprisedb(dot)com>, vignesh C <vignesh21(at)gmail(dot)com>, Peter Smith <smithpb2250(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Markus Wanner <markus(dot)wanner(at)enterprisedb(dot)com>
Subject: Re: locking [user] catalog tables vs 2pc vs logical rep
Date: 2021-06-19 09:51:05
Message-ID: CAA4eK1Kfq+_AFzZWWVrfbSFjhxHfpJ8be0TzhfN5ncG9mbh2NA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Jun 18, 2021 at 2:25 PM osumi(dot)takamichi(at)fujitsu(dot)com
<osumi(dot)takamichi(at)fujitsu(dot)com> wrote:
>
> On Friday, June 18, 2021 11:41 AM osumi(dot)takamichi(at)fujitsu(dot)com <osumi(dot)takamichi(at)fujitsu(dot)com> wrote:

> > Simon, I appreciate your suggestions and yes, if the user catalog table is
> > referenced by the output plugin, it can be another cause of the deadlock.
> >
> > I'm going to post the patch for the those two changes, accordingly.
> Hi, I've made the patch-set to cover the discussion above for all-supported versions.
> Please have a look at those.
>

I have slightly modified your patch, see if the attached looks okay to
you? This is just a HEAD patch, I'll modify the back-branch patches
accordingly.

--
With Regards,
Amit Kapila.

Attachment Content-Type Size
v2-0001-Doc-Update-caveats-in-synchronous-logical-replica.patch application/octet-stream 3.6 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Kapila 2021-06-19 10:00:08 Re: Question about StartLogicalReplication() error path
Previous Message Thomas Munro 2021-06-19 09:37:43 Re: seawasp failing, maybe in glibc allocator