pgsql: Fix pg_receivexlog --slot so that it doesn't prevent the server - Mailing list pgsql-committers

From Fujii Masao
Subject pgsql: Fix pg_receivexlog --slot so that it doesn't prevent the server
Date
Msg-id E1XqxYt-0007iX-Mb@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix pg_receivexlog --slot so that it doesn't prevent the server shutdown.

When pg_receivexlog --slot is connecting to the server, at the shutdown
of the server, walsender keeps waiting for the last WAL record to be
replicated and flushed in pg_receivexlog. But previously pg_receivexlog
issued sync command only when WAL file was switched. So there was
the case where the last WAL was never flushed and walsender had to
keep waiting infinitely. This caused the server shutdown to get stuck.

pg_recvlogical handles this problem by calling fsync() when it receives
the request of immediate reply from the server. That is, at shutdown,
walsender sends the request, pg_recvlogical receives it, flushes the last
WAL record, and sends the flush location back to the server. Since
walsender can see that the last WAL record is successfully flushed, it can
exit cleanly.

This commit introduces the same logic as pg_recvlogical has,
to pg_receivexlog.

Back-patch to 9.4 where pg_receivexlog was changed so that it can use
the replication slot.

Original patch by Michael Paquier, rewritten by me.
Bug report by Furuya Osamu.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/f66c20b317578838a39a1de8014c4363bdc98b9a

Modified Files
--------------
src/bin/pg_basebackup/receivelog.c |   19 +++++++++++++++++++
1 file changed, 19 insertions(+)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Don't require bleeding-edge timezone data in timestamptz regress
Next
From: Fujii Masao
Date:
Subject: pgsql: Fix pg_receivexlog --slot so that it doesn't prevent the server