On Friday, June 11, 2021 2:13 PM vignesh C <vignes...@gmail.com> wrote: > Thanks for the updated patch: > Few comments: > 1) We have used Reordering and Clustering for the same command, we could > rephrase similarly in both places. > + <para> > + Reordering <structname>pg_class</structname> by > <command>CLUSTER</command> > + command in a transaction. > + </para> > > + <para> > + Clustering <structname>pg_trigger</structname> and decoding > <command>PREPARE > + TRANSACTION</command>, if any published table have a trigger > and any > + operations that will be decoded are conducted. > + </para> > + </listitem> > > 2) Here user_catalog_table should be user catalog table > + <para> > + Executing <command>TRUNCATE</command> on > user_catalog_table > in a transaction. > + </para> Thanks for your review.
Attached the patch-set that addressed those two comments.
I also fixed the commit message a bit in the 2PC specific patch to HEAD.
No other changes.
Please check.
Best Regards,
Takamichi Osumi
HEAD_deadlock_documentation_of_logical_decoding_v06.patch
Description: HEAD_deadlock_documentation_of_logical_decoding_v06.patch
HEAD_with_2PC_deadlock_documentation_of_logical_decoding_v06.patch
Description: HEAD_with_2PC_deadlock_documentation_of_logical_decoding_v06.patch
PG10_deadlock_documentation_of_logical_decoding_v06.patch
Description: PG10_deadlock_documentation_of_logical_decoding_v06.patch
PG11_deadlock_documentation_of_logical_decoding_v06.patch
Description: PG11_deadlock_documentation_of_logical_decoding_v06.patch
PG12_deadlock_documentation_of_logical_decoding_v06.patch
Description: PG12_deadlock_documentation_of_logical_decoding_v06.patch
PG13_deadlock_documentation_of_logical_decoding_v06.patch
Description: PG13_deadlock_documentation_of_logical_decoding_v06.patch
PG96_deadlock_documentation_of_logical_decoding_v06.patch
Description: PG96_deadlock_documentation_of_logical_decoding_v06.patch
