diff options
author | Michael Paquier <michael@paquier.xyz> | 2020-07-15 15:17:49 +0900 |
---|---|---|
committer | Michael Paquier <michael@paquier.xyz> | 2020-07-15 15:17:49 +0900 |
commit | 23924feca065aa75f0a9edcf78ad953a51405fe8 (patch) | |
tree | bff4de8808a67f682571f9a732bc8bc853b485f7 /src/common/wait_error.c | |
parent | 4e02f88a4eedd2f9808c16d6d7b50c3a78af2d39 (diff) | |
download | postgresql-23924feca065aa75f0a9edcf78ad953a51405fe8.tar.gz postgresql-23924feca065aa75f0a9edcf78ad953a51405fe8.zip |
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
Diffstat (limited to 'src/common/wait_error.c')
0 files changed, 0 insertions, 0 deletions