Fix handling of missing files when using pg_rewind with online source
authorMichael Paquier <michael@paquier.xyz>
Wed, 15 Jul 2020 06:17:55 +0000 (15:17 +0900)
committerMichael Paquier <michael@paquier.xyz>
Wed, 15 Jul 2020 06:17:55 +0000 (15:17 +0900)
commit14fe8041395cdfaf05c736911080982d021c5e9d
tree06fb4b7402b98db498a1c2299b9f21ee0e3b5533
parentc732df133a3d4ad32b9fb35f2f1b15cb08ac6b3a
Fix handling of missing files when using pg_rewind with online source

When working with an online source cluster, pg_rewind gets a list of all
the files in the source data directory using a WITH RECURSIVE query,
returning a NULL result for a file's metadata if it gets removed between
the moment it is listed in a directory and the moment its metadata is
obtained with pg_stat_file() (say a recycled WAL segment).  The query
result was processed in such a way that for each tuple we checked only
that the first file's metadata was NULL.  This could have two
consequences, both resulting in a failure of the rewind:
- If the first tuple referred to a removed file, all files from the
source would be ignored.
- Any file actually missing would not be considered as such.

While on it, rework slightly the code so as no values are saved if we
know that a file is going to be skipped.

Issue introduced by b36805f, so backpatch down to 9.5.

Author: Justin Pryzby, Michael Paquier
Reviewed-by: Daniel Gustafsson, Masahiko Sawada
Discussion: https://postgr.es/m/20200713061010.GC23581@telsasoft.com
Backpatch-through: 9.5
src/bin/pg_rewind/libpq_fetch.c