Alvaro Herrera ff9f111bce Fix WAL replay in presence of an incomplete record ... Physical replication always ships WAL segment files to replicas once they are complete. This is a problem if one WAL record is split across a segment boundary and the primary server crashes before writing down the segment with the next portion of the WAL record: WAL writing after crash recovery would happily resume at the point where the broken record started, overwriting that record ... but any standby or backup may have already received a copy of that segment, and they are not rewinding. This causes standbys to stop following the primary after the latter crashes: LOG: invalid contrecord length 7262 at A8/D9FFFBC8 because the standby is still trying to read the continuation record (contrecord) for the original long WAL record, but it is not there and it will never be. A workaround is to stop the replica, delete the WAL file, and restart it -- at which point a fresh copy is brought over from the primary. But that's pretty labor intensive, and I bet many users would just give up and re-clone the standby instead. A fix for this problem was already attempted in commit 515e3d84a0, but it only addressed the case for the scenario of WAL archiving, so streaming replication would still be a problem (as well as other things such as taking a filesystem-level backup while the server is down after having crashed), and it had performance scalability problems too; so it had to be reverted. This commit fixes the problem using an approach suggested by Andres Freund, whereby the initial portion(s) of the split-up WAL record are kept, and a special type of WAL record is written where the contrecord was lost, so that WAL replay in the replica knows to skip the broken parts. With this approach, we can continue to stream/archive segment files as soon as they are complete, and replay of the broken records will proceed across the crash point without a hitch. Because a new type of WAL record is added, users should be careful to upgrade standbys first, primaries later. Otherwise they risk the standby being unable to start if the primary happens to write such a record. A new TAP test that exercises this is added, but the portability of it is yet to be seen. This has been wrong since the introduction of physical replication, so backpatch all the way back. In stable branches, keep the new XLogReaderState members at the end of the struct, to avoid an ABI break. Author: Álvaro Herrera <alvherre@alvh.no-ip.org> Reviewed-by: Kyotaro Horiguchi <horikyota.ntt@gmail.com> Reviewed-by: Nathan Bossart <bossartn@amazon.com> Discussion: https://postgr.es/m/202108232252.dh7uxf6oxwcy@alvherre.pgsql		2021-09-29 11:21:51 -03:00
..
backend	Fix WAL replay in presence of an incomplete record	2021-09-29 11:21:51 -03:00
bin	pgbench: Fix handling of socket errors during benchmark.	2021-09-29 21:48:52 +09:00
common	Fix typos and grammar in code comments	2021-09-27 14:21:28 +09:00
fe_utils	Skip trailing whitespaces when parsing integer options	2021-07-27 10:39:05 +09:00
include	Fix WAL replay in presence of an incomplete record	2021-09-29 11:21:51 -03:00
interfaces	Clear conn->errorMessage at successful completion of PQconnectdb().	2021-09-13 16:53:11 -04:00
makefiles	Add NO_INSTALL option to pgxs	2021-05-27 13:58:29 +02:00
pl	Fix misevaluation of STABLE parameters in CALL within plpgsql.	2021-09-21 19:06:53 -04:00
port	windows: Only consider us to be running as service if stderr is invalid.	2021-09-07 11:56:13 -07:00
template	Further tweaking of PG_SYSROOT heuristics for macOS.	2021-01-20 12:07:23 -05:00
test	Fix WAL replay in presence of an incomplete record	2021-09-29 11:21:51 -03:00
timezone	Update time zone data files to tzdata release 2021a.	2021-01-24 16:29:47 -05:00
tools	Fix WAL replay in presence of an incomplete record	2021-09-29 11:21:51 -03:00
tutorial	doc: Prefer explicit JOIN syntax over old implicit syntax in tutorial	2021-04-08 10:51:26 +02:00
.gitignore
DEVELOPERS
Makefile	Remove the option to build thread_test.c outside configure.	2020-10-21 12:08:48 -04:00
Makefile.global.in	Update Unicode data to Unicode 14.0.0	2021-09-15 08:16:44 +02:00
Makefile.shlib	AIX: Fix missing libpq symbols by respecting SHLIB_EXPORTS.	2021-09-06 11:27:59 -07:00
nls-global.mk	Add errhint_plural() function and make use of it	2021-03-31 09:16:25 +02:00