From: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Fix commit timestamp initialization |
Date: | 2015-12-11 17:31:15 |
Message-ID: | E1a7RXH-0001kN-R4@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Fix commit timestamp initialization
This module needs explicit initialization in order to replay WAL records
in recovery, but we had broken this recently following changes to make
other (stranger) scenarios work correctly. To fix, rework the
initialization sequence so that it always takes place before WAL replay
commences for both master and standby.
I could have gone for a more localized fix that just added a "startup"
call for the master server, but it seemed better to restructure the
existing callers as well so that the whole thing made more sense. As a
drawback, there is more control logic in xlog.c now than previously, but
doing otherwise meant passing down the ControlFile flag, which seemed
uglier as a whole.
This also meant adding a check to not re-execute ActivateCommitTs if it
had already been called.
Reported by Fujii Masao.
Backpatch to 9.5.
Branch
------
master
Details
-------
http://git.postgresql.org/pg/commitdiff/69e7235c93e2965cc0e17186bd044e4c54997c19
Modified Files
--------------
src/backend/access/transam/commit_ts.c | 39 ++++++++++++++++++++++----------
src/backend/access/transam/xlog.c | 22 ++++++++++--------
src/include/access/commit_ts.h | 2 +-
3 files changed, 40 insertions(+), 23 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2015-12-11 19:22:36 | pgsql: Still more fixes for planner's handling of LATERAL references. |
Previous Message | Peter Eisentraut | 2015-12-11 03:27:18 | pgsql: Improve some messages |