diff options
| author | Bruce Momjian <bruce@momjian.us> | 2018-08-17 10:25:48 -0400 |
|---|---|---|
| committer | Bruce Momjian <bruce@momjian.us> | 2018-08-17 10:25:48 -0400 |
| commit | 30f480141b646b7ebb86ba1bd880ffa0df503db1 (patch) | |
| tree | 3e635b616894f8a8766e2d29f9995773a5ce7841 /src | |
| parent | 82784f088986140bc08ab22ae34f4b1cbce58eae (diff) | |
| download | postgresql-30f480141b646b7ebb86ba1bd880ffa0df503db1.tar.gz postgresql-30f480141b646b7ebb86ba1bd880ffa0df503db1.zip | |
pg_upgrade: issue helpful error message for use on standbys
Commit 777e6ddf1723306bd2bf8fe6f804863f459b0323 checked for a shut down
message from a standby and allowed it to continue. This patch reports a
helpful error message in these cases, suggesting to use rsync as
documented.
Diagnosed-by: Martín Marqués
Discussion: https://postgr.es/m/CAPdiE1xYCow-reLjrhJ9DqrMu-ppNq0ChUUEvVdxhdjGRD5_eA@mail.gmail.com
Backpatch-through: 9.3
Diffstat (limited to 'src')
| -rw-r--r-- | src/bin/pg_upgrade/controldata.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/src/bin/pg_upgrade/controldata.c b/src/bin/pg_upgrade/controldata.c index 9bf696927b3..69f5c5a2b77 100644 --- a/src/bin/pg_upgrade/controldata.c +++ b/src/bin/pg_upgrade/controldata.c @@ -149,8 +149,14 @@ get_control_data(ClusterInfo *cluster, bool live_check) /* remove leading spaces */ while (*p == ' ') p++; - if (strcmp(p, "shut down\n") != 0 && - strcmp(p, "shut down in recovery\n") != 0) + if (strcmp(p, "shut down in recovery\n") == 0) + { + if (cluster == &old_cluster) + pg_fatal("The source cluster was shut down while in recovery mode. To upgrade, use \"rsync\" as documented or shut it down as a primary.\n"); + else + pg_fatal("The target cluster was shut down while in recovery mode. To upgrade, use \"rsync\" as documented or shut it down as a primary.\n"); + } + else if (strcmp(p, "shut down\n") != 0) { if (cluster == &old_cluster) pg_fatal("The source cluster was not shut down cleanly.\n"); |