Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit882e522

Browse files
Move extra code out of the Pre/PostRestoreCommand() section.
If SIGTERM is received within this section, the startup processwill immediately proc_exit() in the signal handler, so it isinadvisable to include any more code than is required there (assuch code is unlikely to be compatible with doing proc_exit() in asignal handler). This commit moves the code recently added to thissection (see1b06d7b and7fed801) to outside of the section.This ensures that the startup process only calls proc_exit() in itsSIGTERM handler for the duration of the system() call, which is howthis code worked from v8.4 to v14.Reported-by: Michael Paquier, Thomas MunroAnalyzed-by: Andres FreundSuggested-by: Tom LaneReviewed-by: Michael Paquier, Robert Haas, Thomas Munro, Andres FreundDiscussion:https://postgr.es/m/Y9nGDSgIm83FHcad%40paquier.xyzDiscussion:https://postgr.es/m/20230223231503.GA743455%40nathanxps13Backpatch-through: 15
1 parentfdfb141 commit882e522

File tree

1 file changed

+10
-3
lines changed

1 file changed

+10
-3
lines changed

‎src/backend/access/transam/xlogarchive.c

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -161,19 +161,26 @@ RestoreArchivedFile(char *path, const char *xlogfname,
161161
(errmsg_internal("executing restore command \"%s\"",
162162
xlogRestoreCmd)));
163163

164+
pgstat_report_wait_start(WAIT_EVENT_RESTORE_COMMAND);
165+
164166
/*
165-
* Check signals before restore command and reset afterwards.
167+
* PreRestoreCommand() informs the SIGTERM handler for the startup process
168+
* that it should proc_exit() right away. This is done for the duration
169+
* of the system() call because there isn't a good way to break out while
170+
* it is executing. Since we might call proc_exit() in a signal handler,
171+
* it is best to put any additional logic before or after the
172+
* PreRestoreCommand()/PostRestoreCommand() section.
166173
*/
167174
PreRestoreCommand();
168175

169176
/*
170177
* Copy xlog from archival storage to XLOGDIR
171178
*/
172-
pgstat_report_wait_start(WAIT_EVENT_RESTORE_COMMAND);
173179
rc=system(xlogRestoreCmd);
174-
pgstat_report_wait_end();
175180

176181
PostRestoreCommand();
182+
183+
pgstat_report_wait_end();
177184
pfree(xlogRestoreCmd);
178185

179186
if (rc==0)

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp