RE: [PoC] pg_upgrade: allow to upgrade publisher node

From: "Hayato Kuroda (Fujitsu)" <kuroda(dot)hayato(at)fujitsu(dot)com>
To: 'Peter Eisentraut' <peter(dot)eisentraut(at)enterprisedb(dot)com>, 'Julien Rouhaud' <rjuju123(at)gmail(dot)com>
Cc: 'vignesh C' <vignesh21(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
Subject: RE: [PoC] pg_upgrade: allow to upgrade publisher node
Date: 2023-04-26 12:00:08
Message-ID: TYAPR01MB58667E096145842AA8AE6FCEF5659@TYAPR01MB5866.jpnprd01.prod.outlook.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Dear Peter,

> A suggestion: You could write some/most tests against test_decoding
> rather than the publication/subscription system. That way, you can
> avoid many timing issues in the tests and you can check more exactly
> that the slots produce the output you want. This would also help ensure
> that this new facility works for other logical decoding output plugins
> besides the built-in one.

Good point. I think almost tests except --check part can be rewritten.
PSA new patchset.

Additionally, I fixed followings:

- Added initialization for slot_arr.*. This is needed to check whether
the entry has already been allocated, in get_logical_slot_infos().
Previously double-free was occurred in some platform.
- fixed condition in get_logical_slot_infos()
- Changed the expected size of page header to longer one(SizeOfXLogLongPHD).
If the WAL page is the first one in the WAL segment file, the long header seems
to be used.

Best Regards,
Hayato Kuroda
FUJITSU LIMITED

Attachment Content-Type Size
v10-0001-pg_upgrade-Add-include-logical-replication-slots.patch application/octet-stream 33.1 KB
v10-0002-Always-persist-to-disk-logical-slots-during-a-sh.patch application/octet-stream 4.8 KB
v10-0003-pg_upgrade-Add-check-function-for-include-logica.patch application/octet-stream 5.7 KB
v10-0004-Change-the-method-used-to-check-logical-replicat.patch application/octet-stream 7.0 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Aleksander Alekseev 2023-04-26 12:07:18 Re: [PATCH] Extend the length of BackgroundWorker.bgw_library_name
Previous Message David Rowley 2023-04-26 11:48:30 Re: Doc limitation update proposal: include out-of-line OID usage per TOAST-ed columns