Fix handling of missing files when using pg_rewind with online source
authorMichael Paquier <michael@paquier.xyz>
Wed, 15 Jul 2020 06:17:23 +0000 (15:17 +0900)
committerMichael Paquier <michael@paquier.xyz>
Wed, 15 Jul 2020 06:17:23 +0000 (15:17 +0900)
commit1d09fb1f0a9c7edc3298104fc4350e49169f6f2a
tree50e2e38511e3ce7a8425dd59258ef350e17706e1
parente949137397383de58f0c566f5a92b017343c5269
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