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

On Wednesday, June 9, 2021 12:06 PM Amit Kapila  wrote:
> On Tue, Jun 8, 2021 at 6:24 PM vignesh C  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 LOCK on pg_class (or any other catalog table) in a transaction
* Reordering pg_class by CLUSTER command in a transaction
* Executing TRUNCATE 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