From: | "osumi(dot)takamichi(at)fujitsu(dot)com" <osumi(dot)takamichi(at)fujitsu(dot)com> |
---|---|
To: | 'Amit Kapila' <amit(dot)kapila16(at)gmail(dot)com>, vignesh C <vignesh21(at)gmail(dot)com> |
Cc: | 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-09 06:33:14 |
Message-ID: | OSBPR01MB4888771836B71F5BA23B0D1CED369@OSBPR01MB4888.jpnprd01.prod.outlook.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wednesday, June 9, 2021 12:06 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
> On Tue, Jun 8, 2021 at 6:24 PM vignesh C <vignesh21(at)gmail(dot)com> wrote:
> >
> > Thanks for the updated patch.
> >
> > I have few comments:
> > 1) Should we list the actual system tables like pg_class,pg_trigger,
> > etc instead of any other catalog table?
> > User has issued an explicit LOCK on pg_class (or any other catalog
> > table)
> >
>
> I think the way it is mentioned is okay. We don't need to specify other catalog
> tables.
Okay.
> > 2) Here This means deadlock, after this we mention deadlock again for
> > each of the examples, we can remove it if redundant.
> > This can happen in the following ways:
I think this sentence works to notify that commands described below
are major scenarios naturally, to the readers. Then, I don't want to remove it.
If you somehow feel that the descriptions are redundant,
how about unifying all listitems as nouns. like below ?
* An explicit <command>LOCK</command> on <structname>pg_class</structname> (or any other catalog table) in a transaction
* Reordering <structname>pg_class</structname> by <command>CLUSTER</command> command in a transaction
* Executing <command>TRUNCATE</command> on user_catalog_table
> > 3) Should [user] catalog tables be catalog tables or user catalog
> > tables [user] catalog tables
> >
>
> The third point is not clear. Can you please elaborate by quoting the exact
> change from the patch?
IIUC, he means to replace all descriptions "[user] catalog tables"
with "catalog tables or user catalog tables" in the patch,
because seemingly we don't use square brackets to describe optional clause in
normal descriptions(like outside of synopsis and I don't find any example for this).
But, even if so, I would like to keep the current square brackets description,
which makes sentence short and simple.
Best Regards,
Takamichi Osumi
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2021-06-09 06:43:24 | Re: [bug?] Missed parallel safety checks, and wrong parallel safety |
Previous Message | Tom Lane | 2021-06-09 06:28:29 | Re: Fdw batch insert error out when set batch_size > 65535 |