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

Takamichi Osumi (Fujitsu) <osumi.takamichi@fujitsu.com>

From: "osumi.takamichi@fujitsu.com" <osumi.takamichi@fujitsu.com>

To: 'Amit Kapila' <amit.kapila16@gmail.com>, vignesh C <vignesh21@gmail.com>

Cc: Peter Smith <smithpb2250@gmail.com>, Andres Freund <andres@anarazel.de>, PostgreSQL-development <pgsql-hackers@postgresql.org>, Markus Wanner <markus.wanner@enterprisedb.com>

Date: 2021-06-08T08:03:59Z

Lists: pgsql-hackers

Attachments

HEAD_deadlock_documentation_of_logical_decoding_v04.patch (application/octet-stream) patch v4
HEAD_with_2PC_deadlock_documentation_of_logical_decoding_v04.patch (application/octet-stream) patch v4
PG10_deadlock_documentation_of_logical_decoding_v04.patch (application/octet-stream) patch v4
PG11_deadlock_documentation_of_logical_decoding_v04.patch (application/octet-stream) patch v4
PG12_deadlock_documentation_of_logical_decoding_v04.patch (application/octet-stream) patch v4
PG13_deadlock_documentation_of_logical_decoding_v04.patch (application/octet-stream) patch v4
PG96_deadlock_documentation_of_logical_decoding_v04.patch (application/octet-stream) patch v4

On Monday, June 7, 2021 6:22 PM Amit Kapila <amit.kapila16@gmail.com> wrote:
> On Mon, Jun 7, 2021 at 10:44 AM Amit Kapila <amit.kapila16@gmail.com>
> wrote:
> >
> > One more comment is that for HEAD, first just create a patch with
> > synchronous replication-related doc changes and then write a separate
> > patch for prepared transactions.
> >
> 
> I noticed that docs for "Synchronous replication support for Logical Decoding"
> has been introduced by commit
> 49c0864d7ef5227faa24f903902db90e5c9d5d69 which goes till 9.6. So, I think
> you need to create a patch for 9.6 as well unless one of the existing patches
> already applies in 9.6.
OK. I could apply PG10's patch to 9.6.
Also, I've made a separate patch for 2PC description.

On the other hand, I need to mention that
there are some gaps to cause failures to apply patches
between supported versions.
(e.g. applying a patch for HEAD to stable PG13 fails)

To address the gaps between the versions,
I needed to conduct some manual fixes.
Therefore, please note that the content of patch
between PG12 and PG13 are almost same
like PG9.6 and PG10, but, I prepared
independent patches for HEAD and PG11,
in order to make those applied in a comfortable manner.

Kindly have a look at the updated patch-set.
They all passed the test of make html.

Best Regards,
	Takamichi Osumi

Commits

Doc: Update caveats in synchronous logical replication.
- 4be39ef9cbc5 9.6.23 landed
- f365151a5392 10.18 landed
- e00e5db22d79 11.13 landed
- a73bd49c69e1 12.8 landed
- 7a4ecefe9d77 13.4 landed
- c66fb78ebb4f 14.0 landed
Document a few caveats in synchronous logical replication.
- acdb523cb642 9.6.23 landed
- ff33be61f212 10.18 landed
- 5a456034b84e 11.13 landed
- 9f7bba2629cf 13.4 landed
- eb231dbd80d2 12.8 landed
- 3cb828dbe260 14.0 landed
Update the docs and comments for decoding of prepared xacts.
- b4e3dc7fd420 14.0 landed
Allow decoding at prepare time in ReorderBuffer.
- a271a1b50e9b 14.0 cited
Documentation for logical decoding.
- 49c0864d7ef5 9.4.0 cited