diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2020-03-17 16:13:18 -0300 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2020-03-17 16:13:18 -0300 |
| commit | bcd1c3630095e48bc3b1eb0fc8e8c8a7c851eba1 (patch) | |
| tree | d09a3491375f9a49aae88519b0baf601e9bbb64c /src/backend/replication/logical/logicalfuncs.c | |
| parent | 31d846e0265c2c1415d7910d39d5b259b92184ea (diff) | |
| download | postgresql-bcd1c3630095e48bc3b1eb0fc8e8c8a7c851eba1.tar.gz postgresql-bcd1c3630095e48bc3b1eb0fc8e8c8a7c851eba1.zip | |
Fix consistency issues with replication slot copy
Commit 9f06d79ef831's replication slot copying failed to
properly reserve the WAL that the slot is expecting to see
during DecodingContextFindStartpoint (to set the confirmed_flush
LSN), so concurrent activity could remove that WAL and cause the
copy process to error out. But it doesn't actually *need* that
WAL anyway: instead of running decode to find confirmed_flush, it
can be copied from the source slot. Fix this by rearranging things
to avoid DecodingContextFindStartpoint() (leaving the target slot's
confirmed_flush_lsn to invalid), and set that up afterwards by copying
from the target slot's value.
Also ensure the source slot's confirmed_flush_lsn is valid.
Reported-by: Arseny Sher
Author: Masahiko Sawada, Arseny Sher
Discussion: https://postgr.es/m/871rr3ohbo.fsf@ars-thinkpad
Diffstat (limited to 'src/backend/replication/logical/logicalfuncs.c')
0 files changed, 0 insertions, 0 deletions