mirror/postgresql
2
0
Fork 0
mirror of https://git.postgresql.org/git/postgresql.git synced 2024-12-21 08:29:39 +08:00
Code Issues Packages Projects Releases Wiki Activity

Code review of recycling WAL segments in a restartpoint.

Browse Source 
Seems cleaner to get the currently-replayed TLI in the same call to
GetXLogReplayRecPtr that we get the WAL position. Make it more clear in the
comment what the code does when recovery has already ended
(RecoveryInProgress() will set ThisTimeLineID in that case). Finally, make
resetting ThisTimeLineID afterwards more explicit.
This commit is contained in:
Heikki Linnakangas 2013-06-03 09:25:12 +03:00
parent 2c92edad48
commit e1e2bb34f1
1 changed files with 15 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
src/backend/access/transam/xlog.c
View File

@ -7463,6 +7463,7 @@ CreateRestartPoint(int flags)
{ {
XLogRecPtr receivePtr; XLogRecPtr receivePtr;
XLogRecPtr replayPtr; XLogRecPtr replayPtr;
TimeLineID replayTLI;
XLogRecPtr endptr; XLogRecPtr endptr;
/* /*
@ -7470,32 +7471,27 @@ CreateRestartPoint(int flags)
* later. * later.
*/ */
receivePtr = GetWalRcvWriteRecPtr(NULL, NULL); receivePtr = GetWalRcvWriteRecPtr(NULL, NULL);
replayPtr = GetXLogReplayRecPtr(NULL); replayPtr = GetXLogReplayRecPtr(&replayTLI);
endptr = (receivePtr < replayPtr) ? replayPtr : receivePtr; endptr = (receivePtr < replayPtr) ? replayPtr : receivePtr;
KeepLogSeg(endptr, &_logSegNo); KeepLogSeg(endptr, &_logSegNo);
_logSegNo--; _logSegNo--;
/* /*
* Update ThisTimeLineID to the timeline we're currently replaying, so * Try to recycle segments on a useful timeline. If we've been promoted
* that we install any recycled segments on that timeline. * since the beginning of this restartpoint, use the new timeline
* chosen at end of recovery (RecoveryInProgress() sets ThisTimeLineID
* in that case). If we're still in recovery, use the timeline we're
* currently replaying.
* *
* There is no guarantee that the WAL segments will be useful on the * There is no guarantee that the WAL segments will be useful on the
* current timeline; if recovery proceeds to a new timeline right * current timeline; if recovery proceeds to a new timeline right
* after this, the pre-allocated WAL segments on this timeline will * after this, the pre-allocated WAL segments on this timeline will
* not be used, and will go wasted until recycled on the next * not be used, and will go wasted until recycled on the next
* restartpoint. We'll live with that. * restartpoint. We'll live with that.
*
* It's possible or perhaps even likely that we finish recovery while
* a restartpoint is in progress. That means we may get to this point
* some minutes afterwards. Setting ThisTimeLineID at that time would
* actually set it backwards, so we don't want that to persist; if we
* do reset it here, make sure to reset it back afterwards. This
* doesn't look very clean or principled, but its the best of about
* five different ways of handling this edge case.
*/ */
if (RecoveryInProgress()) if (RecoveryInProgress())
(void) GetXLogReplayRecPtr(&ThisTimeLineID); ThisTimeLineID = replayTLI;
RemoveOldXlogFiles(_logSegNo, endptr); RemoveOldXlogFiles(_logSegNo, endptr);
@ -7506,10 +7502,14 @@ CreateRestartPoint(int flags)
PreallocXlogFiles(endptr); PreallocXlogFiles(endptr);
/* /*
* Reset this always, in case we set ThisTimeLineID backwards above. * ThisTimeLineID is normally not set when we're still in recovery.
* Requires no locking; see InitXLOGAccess() * However, recycling/preallocating segments above needed
* ThisTimeLineID to determine which timeline to install the segments
* on. Reset it now, to restore the normal state of affairs for
* debugging purposes.
*/ */
ThisTimeLineID = XLogCtl->ThisTimeLineID; if (RecoveryInProgress())
ThisTimeLineID = 0;
} }
/* /*