diff options
| author | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2016-10-06 13:24:46 +0300 |
|---|---|---|
| committer | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2016-10-06 13:34:38 +0300 |
| commit | aab809664813380b3cd6c94a2791ffddc656b1f1 (patch) | |
| tree | 22cc6ad59cfe59fc7417d023552fa2e790fa9e11 /src | |
| parent | bfcd07b4444d8fdf1ddaba375aa5b3aea419a898 (diff) | |
| download | postgresql-aab809664813380b3cd6c94a2791ffddc656b1f1.tar.gz postgresql-aab809664813380b3cd6c94a2791ffddc656b1f1.zip | |
Disable synchronous commits in pg_rewind.
If you point pg_rewind to a server that is using synchronous replication,
with "pg_rewind --source-server=...", and the replication is not working
for some reason, pg_rewind will get stuck because it creates a temporary
table, which needs to be replicated. You could call broken replication a
pilot error, but pg_rewind is often used in special circumstances, when
there are changes to the replication setup.
We don't do any "real" updates, and we don't care about fsyncing or
replicating the operations on the temporary tables, so fix that by
setting synchronous_commit off.
Michael Banck, Michael Paquier. Backpatch to 9.5, where pg_rewind was
introduced.
Discussion: <20161005143938.GA12247@nighthawk.caipicrew.dd-dns.de>
Diffstat (limited to 'src')
| -rw-r--r-- | src/bin/pg_rewind/libpq_fetch.c | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/src/bin/pg_rewind/libpq_fetch.c b/src/bin/pg_rewind/libpq_fetch.c index 92390099eb0..5128e1c2502 100644 --- a/src/bin/pg_rewind/libpq_fetch.c +++ b/src/bin/pg_rewind/libpq_fetch.c @@ -49,6 +49,7 @@ void libpqConnect(const char *connstr) { char *str; + PGresult *res; conn = PQconnectdb(connstr); if (PQstatus(conn) == CONNECTION_BAD) @@ -77,6 +78,19 @@ libpqConnect(const char *connstr) if (strcmp(str, "on") != 0) pg_fatal("full_page_writes must be enabled in the source server\n"); pg_free(str); + + /* + * Although we don't do any "real" updates, we do work with a temporary + * table. We don't care about synchronous commit for that. It doesn't + * otherwise matter much, but if the server is using synchronous + * replication, and replication isn't working for some reason, we don't + * want to get stuck, waiting for it to start working again. + */ + res = PQexec(conn, "SET synchronous_commit = off"); + if (PQresultStatus(res) != PGRES_COMMAND_OK) + pg_fatal("could not set up connection context: %s", + PQresultErrorMessage(res)); + PQclear(res); } /* |