diff options
| author | Fujii Masao <fujii@postgresql.org> | 2014-08-19 18:30:38 +0900 |
|---|---|---|
| committer | Fujii Masao <fujii@postgresql.org> | 2014-08-19 18:30:38 +0900 |
| commit | c4762886539b06da682045e19ba73dcee3789638 (patch) | |
| tree | 804ecf6fff27078e7f5f4827c84e6fe5aa5dccfd /src/bin/pg_basebackup/pg_basebackup.c | |
| parent | 083d29c65b7897f90c70e6dc0a4240a5fa75c8f2 (diff) | |
| download | postgresql-c4762886539b06da682045e19ba73dcee3789638.tar.gz postgresql-c4762886539b06da682045e19ba73dcee3789638.zip | |
Revert "Fix bug in checking of IDENTIFY_SYSTEM result."
This reverts commit 083d29c65b7897f90c70e6dc0a4240a5fa75c8f2.
The commit changed the code so that it causes an errors when
IDENTIFY_SYSTEM returns three columns. But which prevents us
from using the replication-related utilities against the server
with older version. This is not what we want. For that
compatibility, we allow the utilities to receive three columns
as the result of IDENTIFY_SYSTEM eventhough it actually returns
four columns in 9.4 or later.
Pointed out by Andres Freund.
Diffstat (limited to 'src/bin/pg_basebackup/pg_basebackup.c')
| -rw-r--r-- | src/bin/pg_basebackup/pg_basebackup.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/bin/pg_basebackup/pg_basebackup.c b/src/bin/pg_basebackup/pg_basebackup.c index 3fee38a17d6..3d26e22b8fc 100644 --- a/src/bin/pg_basebackup/pg_basebackup.c +++ b/src/bin/pg_basebackup/pg_basebackup.c @@ -1644,11 +1644,11 @@ BaseBackup(void) progname, "IDENTIFY_SYSTEM", PQerrorMessage(conn)); disconnect_and_exit(1); } - if (PQntuples(res) != 1 || PQnfields(res) < 4) + if (PQntuples(res) != 1 || PQnfields(res) < 3) { fprintf(stderr, _("%s: could not identify system: got %d rows and %d fields, expected %d rows and %d or more fields\n"), - progname, PQntuples(res), PQnfields(res), 1, 4); + progname, PQntuples(res), PQnfields(res), 1, 3); disconnect_and_exit(1); } sysidentifier = pg_strdup(PQgetvalue(res, 0, 0)); |