git.postgresql.org Git - postgresql.git/commitdiff

git projects / postgresql.git / commitdiff

Don't warn about an in-progress online backup, when we're recovering from

author Heikki Linnakangas <heikki.linnakangas@iki.fi>

2010年9月14日 08:05:33 +0000 (08:05 +0000)

committer Heikki Linnakangas <heikki.linnakangas@iki.fi>

2010年9月14日 08:05:33 +0000 (08:05 +0000)

an online backup instead of performing one. pg_ctl can detect that by
checking if recovery.conf exists.

Backup label file is renamed away early in recovery, so the window where
backup label exists during recovery is normally very small, but you can run
into it e.g if restore_command is set incorrectly and the startup process
never finds even the first WAL segment containing the checkpoint record to
start recovery from.

Fujii Masao with comments by me.

src/bin/pg_ctl/pg_ctl.c patch | blob | blame | history

diff --git a/src/bin/pg_ctl/pg_ctl.c b/src/bin/pg_ctl/pg_ctl.c

index 810f6fef6e31dc3e99d65dcc624996cb67d5387f..166a11433426fc0ea57d488b54c1dc96a869a0cb 100644 (file)

--- a/src/bin/pg_ctl/pg_ctl.c

+++ b/src/bin/pg_ctl/pg_ctl.c

@@ -4,7 +4,7 @@

- * $PostgreSQL: pgsql/src/bin/pg_ctl/pg_ctl.c,v 1.122 2010年04月07日 03:48:51 itagaki Exp $

+ * $PostgreSQL: pgsql/src/bin/pg_ctl/pg_ctl.c,v 1.123 2010年09月14日 08:05:33 heikki Exp $

*-------------------------------------------------------------------------

@@ -142,6 +142,7 @@ static char postopts_file[MAXPGPATH];

static char pid_file[MAXPGPATH];

static char conf_file[MAXPGPATH];

static char backup_file[MAXPGPATH];

+static char recovery_file[MAXPGPATH];

#if defined(HAVE_GETRLIMIT) && defined(RLIMIT_CORE)

static void unlimit_core_size(void);

@@ -802,7 +803,15 @@ do_stop(void)

}

else

{

- if ((shutdown_mode == SMART_MODE) && (stat(backup_file, &statbuf) == 0))

+ /*

+ * If backup_label exists, an online backup is running. Warn the

+ * user that smart shutdown will wait for it to finish. However, if

+ * recovery.conf is also present, we're recovering from an online

+ * backup instead of performing one.

+ */

+ if (shutdown_mode == SMART_MODE &&

+ stat(backup_file, &statbuf) == 0 &&

+ stat(recovery_file, &statbuf) != 0)

{

print_msg(_("WARNING: online backup mode is active\n"

"Shutdown will not complete until pg_stop_backup() is called.\n\n"));

@@ -879,7 +888,15 @@ do_restart(void)

exit(1);

}

- if ((shutdown_mode == SMART_MODE) && (stat(backup_file, &statbuf) == 0))

+ /*

+ * If backup_label exists, an online backup is running. Warn the

+ * user that smart shutdown will wait for it to finish. However, if

+ * recovery.conf is also present, we're recovering from an online

+ * backup instead of performing one.

+ */

+ if (shutdown_mode == SMART_MODE &&

+ stat(backup_file, &statbuf) == 0 &&

+ stat(recovery_file, &statbuf) != 0)

{

print_msg(_("WARNING: online backup mode is active\n"

"Shutdown will not complete until pg_stop_backup() is called.\n\n"));

@@ -1962,6 +1979,7 @@ main(int argc, char **argv)

snprintf(pid_file, MAXPGPATH, "%s/postmaster.pid", pg_data);

snprintf(conf_file, MAXPGPATH, "%s/postgresql.conf", pg_data);

snprintf(backup_file, MAXPGPATH, "%s/backup_label", pg_data);

+ snprintf(recovery_file, MAXPGPATH, "%s/recovery.conf", pg_data);

}

switch (ctl_command)

This is the main PostgreSQL git repository.

RSS Atom