From: | "Hayato Kuroda (Fujitsu)" <kuroda(dot)hayato(at)fujitsu(dot)com> |
---|---|
To: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Cc: | Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>, Julien Rouhaud <rjuju123(at)gmail(dot)com>, Sawada Masahiko <sawada(dot)mshk(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, 'vignesh C' <vignesh21(at)gmail(dot)com> |
Subject: | RE: persist logical slots to disk during shutdown checkpoint |
Date: | 2023-08-28 11:48:14 |
Message-ID: | TYAPR01MB586612783A24C51A16831713F5E0A@TYAPR01MB5866.jpnprd01.prod.outlook.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Dear hackers,
I also tested for logical slots on the physical standby. PSA the script.
confirmed_flush_lsn for such slots were successfully persisted.
# Topology
In this test nodes are connected each other.
node1 --(physical replication)-->node2--(logical replication)-->node3
# Test method
An attached script did following steps
1. constructed above configurations
2. Inserted data on node1
3. read confirmed_flush_lsn on node2 (a)
4. restarted node2
5. read confirmed_flush_lsn again on node2 (b)
6. compare (a) and (b)
# result
Before patching, (a) and (b) were different value, which meant that logical
slots on physical standby were not saved at shutdown.
```
slot_name | confirmed_flush_lsn
-----------+---------------------
sub | 0/30003E8
(1 row)
waiting for server to shut down.... done
server stopped
waiting for server to start.... done
server started
slot_name | confirmed_flush_lsn
-----------+---------------------
sub | 0/30000D8
(1 row)
```
After patching, (a) and (b) became the same value. The v4 patch worked well even
if the node is physical standby.
```
slot_name | confirmed_flush_lsn
-----------+---------------------
sub | 0/30003E8
(1 row)
waiting for server to shut down.... done
server stopped
waiting for server to start.... done
server started
slot_name | confirmed_flush_lsn
-----------+---------------------
sub | 0/30003E8
(1 row)
```
Best Regards,
Hayato Kuroda
FUJITSU LIMITED
Attachment | Content-Type | Size |
---|---|---|
test_for_physical_standby.sh | application/octet-stream | 2.2 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Pavel Stehule | 2023-08-28 11:58:55 | Re: proposal: psql: show current user in prompt |
Previous Message | Aleksander Alekseev | 2023-08-28 11:48:05 | Re: [PATCH v1] PQputCopyEnd never returns 0, fix the inaccurate comment |