pgsql: Add test case for an archive recovery corner case. - Mailing list pgsql-committers

From Heikki Linnakangas
Subject pgsql: Add test case for an archive recovery corner case.
Date
Msg-id E1nJXkI-0004VV-C3@gemulon.postgresql.org
Whole thread Raw
Responses Re: pgsql: Add test case for an archive recovery corner case.
List pgsql-committers
Add test case for an archive recovery corner case.

While I was working on a patch to refactor things around xlog.c, I mixed
up EndOfLogTLI and replayTLI at the end of recovery. As a result, if you
recovered to a point with a lower-numbered timeline in a WAL segment
that has a higher TLI in the filename, the end-of-recovery WAL record
was created with invalid PrevTimeLineId. I noticed that while
self-reviewing, but no tests failed. So add a test to cover that corner
case.

Thanks to Amul Sul who also submitted a test case for the same corner
case, although this patch is different from that.

Reviewed-by: Amul Sul, Michael Paquier
Discussion: https://www.postgresql.org/message-id/52bc9ccd-8591-431b-0086-15d9acf25a3f@iki.fi
Discussion:
https://www.postgresql.org/message-id/CAAJ_b94Vjt5cXGza_1MkjLQWciNdEemsmiWuQj0d%3DM7JfjAa1g%40mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/50e5bc582a4371a88218078a2c651d901bf87d96

Modified Files
--------------
src/test/recovery/t/028_pitr_timelines.pl | 176 ++++++++++++++++++++++++++++++
1 file changed, 176 insertions(+)


pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: pgsql: Database-level collation version tracking
Next
From: Tomas Vondra
Date:
Subject: Re: pgsql: Fix skip-empty-xacts with sequences in test_decoding